Patchwork Fix g++.dg/opt/pr56999.C

login
register
mail settings
Submitter Jakub Jelinek
Date March 24, 2014, 6:21 a.m.
Message ID <20140324062105.GZ1817@tucnak.redhat.com>
Download mbox | patch
Permalink /patch/332964/
State New
Headers show

Comments

Jakub Jelinek - March 24, 2014, 6:21 a.m.
On Sun, Mar 23, 2014 at 09:47:21PM -0400, John David Anglin wrote:
> On hppa64-hp-hpux11.11, the lack of a pragma to pop the currently
> hidden visibility before
> main results in main being hidden.  The attached change adds a pop
> pragma to resolve the
> test failure:
> ld: (Warning) Potential unresolved symbol "main": shared library
> reference is resolved to non-exported definition in file "/var/tmp//
> ccUra9KR.o". FAIL: g++.dg/opt/pr56999.C -std=gnu++98 (test for
> excess errors)
> Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11.
> 
> OK for trunk?

2014-03-23  John David Anglin  <danglin@gcc.gnu.org>

	PR testsuite/58013
	* g++.dg/opt/pr56999.C: Pop hidden visibility.


Ok with that change (just verified the testcase is still miscompiled with that change
before Vlad's r198082 fix and works after it.

	Jakub

Patch

Index: g++.dg/opt/pr56999.C
===================================================================
--- g++.dg/opt/pr56999.C	(revision 208769)
+++ g++.dg/opt/pr56999.C	(working copy)
@@ -159,6 +159,7 @@ 
     pendingRecompiles->append (info);
 }
 volatile JITScript *JSScript::JITScriptHandle::UNJITTABLE;
+#pragma GCC visibility pop(hidden)

This must be
#pragma GCC visibility pop