tic54x fixes

A number of the tic54x tests were failing, and I thought it worth
investigating since the target makes use of a lot of hash tables, and
we've just changed gas hash tables.  It turns out none of the gas
failures were due to hashing.

	* config/tc-tic54x.c (stag_add_field_symbols): Don't free "name"
	in case where it isn't copied.
	* config/tc-tic54x.h (LOCAL_LABELS_FB): Undef.
	* testsuite/gas/tic54x/field.d: Dump section contents and symbols
	rather than disassembling.
	* testsuite/gas/tic54x/set.d: Adjust for newer disassembly.

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 6b634a0f360df5e94812d4a620474ae703e1bb36..60114ed2226dc607a9700e7661407ec2124446bd 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,12 @@
+2020-08-24  Alan Modra  <amodra@gmail.com>
+
+       * config/tc-tic54x.c (stag_add_field_symbols): Don't free "name"
+       in case where it isn't copied.
+       * config/tc-tic54x.h (LOCAL_LABELS_FB): Undef.
+       * testsuite/gas/tic54x/field.d: Dump section contents and symbols
+       rather than disassembling.
+       * testsuite/gas/tic54x/set.d: Adjust for newer disassembly.
+
 2020-08-24  Alan Modra  <amodra@gmail.com>
 
        * config/tc-aarch64.c (md_begin): Don't bother checking for

diff --git a/gas/config/tc-tic54x.c b/gas/config/tc-tic54x.c
index 62d49d9bc8b0b011d0c48459d38f42eb4c2d116f..5fe1b2f92ea0779e5465dd8077c469812095abae 100644 (file)
--- a/gas/config/tc-tic54x.c
+++ b/gas/config/tc-tic54x.c
@@ -551,6 +551,7 @@ stag_add_field_symbols (struct stag *stag,
   while (field != NULL)
     {
       char *name = concat (prefix, field->name, NULL);
+      char *freename = name;
 
       if (rootsym == NULL)
        {
@@ -568,6 +569,7 @@ stag_add_field_symbols (struct stag *stag,
          replacement = concat (S_GET_NAME (rootsym), "+", root_stag_name,
                                name + strlen (S_GET_NAME (rootsym)), NULL);
          str_hash_insert (subsym_hash[0], name, replacement, 0);
+         freename = NULL;
        }
 
       /* Recurse if the field is a structure.
@@ -577,7 +579,7 @@ stag_add_field_symbols (struct stag *stag,
                                field->offset,
                                rootsym, root_stag_name);
       field = field->next;
-      free (name);
+      free (freename);
     }
   free (prefix);
 }

diff --git a/gas/config/tc-tic54x.h b/gas/config/tc-tic54x.h
index 8d0d0f195d65dab220f43480a8a5af6c39be6c9a..5ba4a6ad5e805ababe8834117bb1dd723f09a26b 100644 (file)
--- a/gas/config/tc-tic54x.h
+++ b/gas/config/tc-tic54x.h
@@ -35,6 +35,8 @@
 #define MAX_OPERANDS 4
 #define PARALLEL_SEPARATOR '|'
 #define LABELS_WITHOUT_COLONS 1
+#undef LOCAL_LABELS_FB
+
 /* accept 0FFFFh, 1010b, etc.  */
 #define NUMBERS_WITH_SUFFIX 1
 /* $ is section program counter */

diff --git a/gas/testsuite/gas/tic54x/field.d b/gas/testsuite/gas/tic54x/field.d
index 60a4868904cd1aef65930baf772654588d6742dc..941d36b28a6b4c10977b9499281c13a767a4f931 100644 (file)
--- a/gas/testsuite/gas/tic54x/field.d
+++ b/gas/testsuite/gas/tic54x/field.d
@@ -1,30 +1,19 @@
-#objdump: -d
+#objdump: -s -t
 #name: c54x field directive
 
-.*: +file format .*c54x.*
-
-Disassembly of section .text:
-
-0+000 <f1>:
-   0:  2af0.*
-   1:  5600.*
-
-0+001 <f2>:
-   1:  5600.*
-
-0+002 <f4>:
-   2:  0001.*
-
-0+003 <f5>:
-   3:  0000.*
-   4:  4321.*
-
-0+005 <f6>:
-   5:  000f.*
-
-0+006 <f7>:
-   6:  6000.*
-   7:  008a.*
-
-0+007 <f8>:
-   7:  008a.*
+.*:     file format .*
+
+SYMBOL TABLE:
+#...
+.* 0x0+0 f1
+.* 0x0+1 f2
+.* 0x0+1 f3
+.* 0x0+2 f4
+.* 0x0+3 f5
+.* 0x0+5 f6
+.* 0x0+6 f7
+.* 0x0+7 f8
+#...
+
+Contents of section \.text:
+ 0000 f02a0056 01000000 21430f00 00608a00 .*

diff --git a/gas/testsuite/gas/tic54x/set.d b/gas/testsuite/gas/tic54x/set.d
index 8f661884d43456d125a597d8da500a874586f2f8..1fd69bfa4fd034d28bd3404ff65da7fb65fb0862 100644 (file)
--- a/gas/testsuite/gas/tic54x/set.d
+++ b/gas/testsuite/gas/tic54x/set.d
@@ -13,4 +13,6 @@ Disassembly of section .text:
 
 0+004 <LABEL>:
    4:  000a.*
+
+0+005 <SYMTAB>:
    5:  0035.*