Patchwork [PR,55238] More careful pass-through handling in find_aggregate_values_for_callers_subset

login
register
mail settings
Submitter Martin Jambor
Date Nov. 20, 2012, 1:31 p.m.
Message ID <20121120133144.GB14889@virgil.arch.suse.de>
Download mbox | patch
Permalink /patch/200333/
State New
Headers show

Comments

Martin Jambor - Nov. 20, 2012, 1:31 p.m.
Hi,

On Fri, Nov 16, 2012 at 08:42:46AM -0500, David Edelsohn wrote:
> 	* testsuite/gcc.dg/torture/pr55238.c: New test.
> 
> Does this PR rely on hidden visibility?  The new testcase uses
> attribute visibility hidden, but does not check DejaGNU
> dg-require-visibility and fails on AIX.

No, at least on my i686 desktop it does not rely on it.  I will apply
the following patch in a few hours (after I add it to some testing run
on x86_64), unless someone objects.

Thanks,

Martin


2012-11-20  Martin Jambor  <mjambor@suse.cz>

	* gcc.dg/torture/pr55238.c: Remove hidden attribute.

Patch

Index: src/gcc/testsuite/gcc.dg/torture/pr55238.c
===================================================================
--- src.orig/gcc/testsuite/gcc.dg/torture/pr55238.c
+++ src/gcc/testsuite/gcc.dg/torture/pr55238.c
@@ -31,7 +31,7 @@  gzbuffer (void *file, int size)
   gz_reset (state);
 }
 
-void __attribute__ ((visibility ("hidden"))) gz_error (gz_state *state, int err, char *msg)
+void gz_error (gz_state *state, int err, char *msg)
 {
   if (state->msg != 0)
     {