Tidy up some C style

- SymbolValue / SetSymbolValue should take lispobj, not u64.

- pseudo-atomic must explicitly depend on the header that declares "lose()"
  otherwise the C compiler could warn "did you mean close?" differing
  based on whether some other header included "interr.h"
  (And "interr" depends on the header that defines the preprocessor macro
  'never_returns'. It's non-self-containedness all the way down)

src/runtime/pseudo-atomic.h

@@ -16,6 +16,8 @@
 #ifndef PSEUDO_ATOMIC_H
 #define PSEUDO_ATOMIC_H
 
+#include "interr.h"
+
 #if defined(LISP_FEATURE_X86) || defined(LISP_FEATURE_X86_64)
 
 #define set_alloc_pointer(value) dynamic_space_free_pointer = (lispobj*)(value)

src/runtime/thread.h

@@ -110,9 +110,9 @@ tls_index_of(struct symbol *symbol) // untagged pointer
 #endif
 
 static inline lispobj
-SymbolValue(u64 tagged_symbol_pointer, void *thread)
+SymbolValue(lispobj tagged_symbol_pointer, void *thread)
 {
- struct symbol *sym= SYMBOL(tagged_symbol_pointer);
+ struct symbol *sym = SYMBOL(tagged_symbol_pointer);
  if(thread && tls_index_of(sym)) {
  lispobj r = per_thread_value(sym, thread);
  if(r!=NO_TLS_VALUE_MARKER_WIDETAG) return r;
@@ -121,9 +121,9 @@ SymbolValue(u64 tagged_symbol_pointer, void *thread)
 }
 
 static inline void
-SetSymbolValue(u64 tagged_symbol_pointer,lispobj val, void *thread)
+SetSymbolValue(lispobj tagged_symbol_pointer,lispobj val, void *thread)
 {
- struct symbol *sym= SYMBOL(tagged_symbol_pointer);
+ struct symbol *sym = SYMBOL(tagged_symbol_pointer);
  if(thread && tls_index_of(sym)) {
  if (per_thread_value(sym, thread) != NO_TLS_VALUE_MARKER_WIDETAG) {
  per_thread_value(sym, thread) = val;