diff mbox

Omit AIX mapping class from VAR_DECL

Message ID CAGWvny=b9O1WJ+xggs=XhYJ75t_yJhysmJkavsT1FA07Mda7oQ@mail.gmail.com
State New
Headers show

Commit Message

David Edelsohn Sept. 27, 2016, 2 p.m. UTC 
AIX Assembler uses storage mapping classes to map symbols and CSECTs
to XCOFF file sections. References to symbols with a section to be
determined in the future are suppose to use [UA] storage mapping class
for "unclassified".

rs6000.c:rs6000_output_symbol() adds the [DS] decoration for function
symbols and recently added the [UA] decoration for extern data.
rs6000_output_symbol() both emitted the decoration and also changed
the string in the DECL.  For data symbols, this causes a failure in
some of the GCC hashing and comparison routines.

This patch adjusts rs6000_output_symbol() to only modify the DECL for
FUNCTION_DECLs not VAR_DECLs.

Bootstrapped on powrepc-ibm-aix7.1.0.0

* config/rs6000/rs6000.c (rs6000_output_symbol): Don't modify VAR_DECL string.
diff mbox

Patch

Index: rs6000.c
===================================================================
--- rs6000.c    (revision 240516)
+++ rs6000.c    (working copy)
@@ -30309,7 +30309,10 @@  rs6000_output_symbol_ref (FILE *file, rtx x)
                     (TREE_CODE (decl) == FUNCTION_DECL
                      ? "[DS]" : "[UA]"),
                     NULL);
-      XSTR (x, 0) = name;
+
+      /* Don't modify name in extern VAR_DECL to include mapping class.  */
+      if (TREE_CODE (decl) == FUNCTION_DECL)
+       XSTR (x, 0) = name;
     }

   if (VTABLE_NAME_P (name))