Re: [PATCH] Fix g++.dg/opt/pr56999.C

2014-03-24 Thread Jakub Jelinek
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  dang...@gcc.gnu.org

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

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

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] Fix g++.dg/opt/pr56999.C

2014-03-23 Thread John David Anglin
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?

Dave
--
John David Anglin   dave.ang...@bell.net




pr56999.C.d
Description: Binary data