* [PATCH] Omit AIX mapping class from VAR_DECL
@ 2016-09-27 14:02 David Edelsohn
0 siblings, 0 replies; only message in thread
From: David Edelsohn @ 2016-09-27 14:02 UTC (permalink / raw)
To: GCC Patches
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.
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))
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2016-09-27 14:00 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-27 14:02 [PATCH] Omit AIX mapping class from VAR_DECL David Edelsohn
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).