This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] Fix gcc.dg/visibility-1{4,6}.c failures on s390-linux


Hi!

These fail because foo is output through s390_output_pool_entry, not
using expand_expr* nor output_operand, and so assemble_external isn't
called on the referenced symbols, which means .hidden foo isn't emitted,
eventhough the hidden symbol is referenced.

Ok for trunk if it passes bootstrap/regtest?

2008-10-30  Jakub Jelinek  <jakub@redhat.com>

	* config/s390/s390.c (s390_mark_symbol_ref_as_used): New function.
	(s390_output_pool_entry): Call it through for_each_rtx.

--- gcc/config/s390/s390.c.jj	2008-10-14 12:49:25.000000000 +0200
+++ gcc/config/s390/s390.c	2008-10-30 11:50:59.000000000 +0100
@@ -6664,6 +6664,24 @@ s390_chunkify_cancel (struct constant_po
     }
 }
 
+/* Helper rtx-iteration-function for s390_output_pool_entry.  Marks
+   SYMBOL_REFs as referenced through use of assemble_external.  */
+
+static int
+s390_mark_symbol_ref_as_used (rtx *x, void *dummy ATTRIBUTE_UNUSED)
+{
+  /* If we have a used symbol, we may have to emit assembly
+     annotations corresponding to whether the symbol is external, weak
+     or has non-default visibility.  */
+  if (GET_CODE (*x) == SYMBOL_REF)
+    {
+      tree t = SYMBOL_REF_DECL (*x);
+      if (t)
+        assemble_external (t);
+      return -1;
+    }
+  return 0;
+}
 
 /* Output the constant pool entry EXP in mode MODE with alignment ALIGN.  */
 
@@ -6684,6 +6702,7 @@ s390_output_pool_entry (rtx exp, enum ma
 
     case MODE_INT:
       assemble_integer (exp, GET_MODE_SIZE (mode), align, 1);
+      for_each_rtx (&exp, s390_mark_symbol_ref_as_used, NULL);
       break;
 
     default:

	Jakub


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]