Wed Jul 13 18:01:58 1994 Ken Raeburn (raeburn@cujo.cygnus.com)
[deliverable/binutils-gdb.git] / gas / symbols.c
index c66f48ba6aeb2aac3f6d0317bdd6c08fb8ecba78..c177902d7656d94e5cba6f2b4180e8dd1477acd8 100644 (file)
@@ -17,7 +17,7 @@
    along with GAS; see the file COPYING.  If not, write to
    the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
-/* #define DEBUG_SYMS /* to debug symbol list maintenance */
+/* #define DEBUG_SYMS / * to debug symbol list maintenance */
 
 #include <ctype.h>
 
@@ -38,32 +38,10 @@ symbolS *symbol_rootP;
 symbolS *symbol_lastP;
 symbolS abs_symbol;
 
-symbolS *dot_text_symbol;
-symbolS *dot_data_symbol;
-symbolS *dot_bss_symbol;
-
 struct obstack notes;
 
 static void fb_label_init PARAMS ((void));
 
-void
-symbol_begin ()
-{
-  symbol_lastP = NULL;
-  symbol_rootP = NULL;         /* In case we have 0 symbols (!!) */
-  sy_hash = hash_new ();
-  memset ((char *) (&abs_symbol), '\0', sizeof (abs_symbol));
-#ifdef BFD_ASSEMBLER
-  abs_symbol.bsym = bfd_abs_section.symbol;
-#else
-  /* Can't initialise a union. Sigh. */
-  S_SET_SEGMENT (&abs_symbol, absolute_section);
-#endif
-#ifdef LOCAL_LABELS_FB
-  fb_label_init ();
-#endif /* LOCAL_LABELS_FB */
-}
-
 /*
  *                     symbol_new()
  *
@@ -82,10 +60,10 @@ symbol_begin ()
  */
 
 symbolS *
-symbol_new (name, segment, value, frag)
+symbol_new (name, segment, valu, frag)
      CONST char *name;         /* It is copied, the caller can destroy/modify */
      segT segment;             /* Segment identifier (SEG_<something>) */
-     valueT value;             /* Symbol value */
+     valueT valu             /* Symbol value */
      fragS *frag;              /* Associated fragment */
 {
   unsigned int name_length;
@@ -113,13 +91,13 @@ symbol_new (name, segment, value, frag)
   S_SET_NAME (symbolP, preserved_copy_of_name);
 
   S_SET_SEGMENT (symbolP, segment);
-  S_SET_VALUE (symbolP, value);
+  S_SET_VALUE (symbolP, valu);
   symbol_clear_list_pointers(symbolP);
 
   symbolP->sy_frag = frag;
 #ifndef BFD_ASSEMBLER
   symbolP->sy_number = ~0;
-  symbolP->sy_name_offset = ~0;
+  symbolP->sy_name_offset = (unsigned int) ~0;
 #endif
 
   /*
@@ -281,10 +259,10 @@ colon (sym_name)          /* just seen "x:" - rattle symbols & frags */
                            S_GET_OTHER (symbolP), S_GET_DESC (symbolP),
                            (long) S_GET_VALUE (symbolP));
 #else
-                 as_fatal ("Symbol \"%s\" is already defined as \"%s\"/%d.",
+                 as_fatal ("Symbol \"%s\" is already defined as \"%s\"/%ld.",
                            sym_name,
                            segment_name (S_GET_SEGMENT (symbolP)),
-                           S_GET_VALUE (symbolP));
+                           (long) S_GET_VALUE (symbolP));
 #endif
                }
            }                   /* if the undefined symbol has no value */
@@ -329,12 +307,12 @@ void
 symbol_table_insert (symbolP)
      symbolS *symbolP;
 {
-  register char *error_string;
+  register const char *error_string;
 
   know (symbolP);
   know (S_GET_NAME (symbolP));
 
-  if (*(error_string = hash_jam (sy_hash, S_GET_NAME (symbolP), (char *) symbolP)))
+  if ((error_string = hash_jam (sy_hash, S_GET_NAME (symbolP), (PTR) symbolP)))
     {
       as_fatal ("Inserting \"%s\" into symbol table failed: %s",
                S_GET_NAME (symbolP), error_string);
@@ -542,7 +520,7 @@ verify_symbol_chain (rootP, lastP)
       know (symbolP->sy_next->sy_previous == symbolP);
 #else
       /* Walk the list anyways, to make sure pointers are still good.  */
-      *symbolP;
+      ;
 #endif /* SYMBOLS_NEED_BACKPOINTERS */
     }
 
@@ -571,14 +549,19 @@ void
 resolve_symbol_value (symp)
      symbolS *symp;
 {
+  int resolved;
+
   if (symp->sy_resolved)
     return;
 
+  resolved = 0;
+
   if (symp->sy_resolving)
     {
       as_bad ("Symbol definition loop encountered at %s",
              S_GET_NAME (symp));
       S_SET_VALUE (symp, (valueT) 0);
+      resolved = 1;
     }
   else
     {
@@ -596,6 +579,7 @@ resolve_symbol_value (symp)
          S_SET_VALUE (symp, S_GET_VALUE (symp) + symp->sy_frag->fr_address);
          if (S_GET_SEGMENT (symp) == expr_section)
            S_SET_SEGMENT (symp, absolute_section);
+         resolved = 1;
          break;
 
        case O_symbol:
@@ -614,6 +598,8 @@ resolve_symbol_value (symp)
              || S_GET_SEGMENT (symp) == undefined_section)
            S_SET_SEGMENT (symp,
                           S_GET_SEGMENT (symp->sy_value.X_add_symbol));
+
+         resolved = symp->sy_value.X_add_symbol->sy_resolved;
          break;
 
        case O_uminus:
@@ -630,6 +616,7 @@ resolve_symbol_value (symp)
          if (S_GET_SEGMENT (symp) == expr_section
              || S_GET_SEGMENT (symp) == undefined_section)
            S_SET_SEGMENT (symp, absolute_section);
+         resolved = symp->sy_value.X_add_symbol->sy_resolved;
          break;
 
        case O_multiply:
@@ -681,17 +668,30 @@ resolve_symbol_value (symp)
          if (S_GET_SEGMENT (symp) == expr_section
              || S_GET_SEGMENT (symp) == undefined_section)
            S_SET_SEGMENT (symp, absolute_section);
+         resolved = (symp->sy_value.X_add_symbol->sy_resolved
+                     && symp->sy_value.X_op_symbol->sy_resolved);
          break;
 
        case O_register:
        case O_big:
        case O_illegal:
-         as_bad ("bad value for symbol \"%s\"", S_GET_NAME (symp));
+         /* Give an error (below) if not in expr_section.  We don't
+            want to worry about expr_section symbols, because they
+            are fictional (they are created as part of expression
+            resolution), and any problems may not actually mean
+            anything.  */
          break;
        }
     }
 
-  symp->sy_resolved = 1;
+  /* Don't worry if we can't resolve an expr_section symbol.  */
+  if (resolved)
+    symp->sy_resolved = 1;
+  else if (S_GET_SEGMENT (symp) != expr_section)
+    {
+      as_bad ("can't resolve value for symbol \"%s\"", S_GET_NAME (symp));
+      symp->sy_resolved = 1;
+    }
 }
 
 #ifdef LOCAL_LABELS_DOLLAR
@@ -707,7 +707,7 @@ static long *dollar_labels;
 static long *dollar_label_instances;
 static char *dollar_label_defines;
 static long dollar_label_count;
-static long dollar_label_max;
+static unsigned long dollar_label_max;
 
 int 
 dollar_label_defined (label)
@@ -745,7 +745,7 @@ dollar_label_instance (label)
 void 
 dollar_label_clear ()
 {
-  memset (dollar_label_defines, '\0', dollar_label_count);
+  memset (dollar_label_defines, '\0', (unsigned int) dollar_label_count);
 }
 
 #define DOLLAR_LABEL_BUMP_BY 10
@@ -874,8 +874,8 @@ dollar_label_name (n, augend)
 static long fb_low_counter[FB_LABEL_SPECIAL];
 static long *fb_labels;
 static long *fb_label_instances;
-static long fb_label_count = 0;
-static long fb_label_max = 0;
+static long fb_label_count;
+static long fb_label_max;
 
 /* this must be more than FB_LABEL_SPECIAL */
 #define FB_LABEL_BUMP_BY (FB_LABEL_SPECIAL + 6)
@@ -1093,6 +1093,7 @@ S_SET_VALUE (s, val)
 {
   s->sy_value.X_op = O_constant;
   s->sy_value.X_add_number = (offsetT) val;
+  s->sy_value.X_unsigned = 0;
 }
 
 #ifdef BFD_ASSEMBLER
@@ -1104,17 +1105,17 @@ S_IS_EXTERNAL (s)
   flagword flags = s->bsym->flags;
 
   /* sanity check */
-  if (flags & BSF_LOCAL && flags & (BSF_EXPORT | BSF_GLOBAL))
+  if (flags & BSF_LOCAL && flags & BSF_GLOBAL)
     abort ();
 
-  return (flags & (BSF_EXPORT | BSF_GLOBAL)) != 0;
+  return (flags & BSF_GLOBAL) != 0;
 }
 
 int
 S_IS_COMMON (s)
      symbolS *s;
 {
-  return s->bsym->section == &bfd_com_section;
+  return bfd_is_com_section (s->bsym->section);
 }
 
 int
@@ -1140,7 +1141,7 @@ S_IS_LOCAL (s)
   flagword flags = s->bsym->flags;
 
   /* sanity check */
-  if (flags & BSF_LOCAL && flags & (BSF_EXPORT | BSF_GLOBAL))
+  if (flags & BSF_LOCAL && flags & BSF_GLOBAL)
     abort ();
 
   return (S_GET_NAME (s)
@@ -1220,4 +1221,22 @@ S_SET_NAME (s, name)
 }
 #endif /* BFD_ASSEMBLER */
 
+void
+symbol_begin ()
+{
+  symbol_lastP = NULL;
+  symbol_rootP = NULL;         /* In case we have 0 symbols (!!) */
+  sy_hash = hash_new ();
+  memset ((char *) (&abs_symbol), '\0', sizeof (abs_symbol));
+#ifdef BFD_ASSEMBLER
+  abs_symbol.bsym = bfd_abs_section.symbol;
+#else
+  /* Can't initialise a union. Sigh. */
+  S_SET_SEGMENT (&abs_symbol, absolute_section);
+#endif
+#ifdef LOCAL_LABELS_FB
+  fb_label_init ();
+#endif /* LOCAL_LABELS_FB */
+}
+
 /* end of symbols.c */
This page took 0.026364 seconds and 4 git commands to generate.