Hi Martin, On 2019-02-22 5:21 a.m., Martin Liška wrote: > Hi. > > This patch makes main not hidden (as reported in the PR). > John: Can you verify it fixes your problem on hppa64-hp-hpux11.11 target? Your patch fixed the problem on the hppa64-hp-hpux11.11 target but we also need to require visibility support on 32-bit hppa-hpux. See attached.
Dave -- John David Anglin dave.ang...@bell.net
Index: gcc/testsuite/g++.dg/ipa/pr89009.C =================================================================== --- gcc/testsuite/g++.dg/ipa/pr89009.C (revision 268997) +++ gcc/testsuite/g++.dg/ipa/pr89009.C (working copy) @@ -1,11 +1,12 @@ /* PR ipa/89009 */ /* { dg-do run } */ -/* { dg-options "-fvisibility=hidden -fpic -O2 -fno-inline" } */ +/* { dg-options "-fpic -O2 -fno-inline" } */ +/* { dg-require-visibility "" } */ -#pragma GCC visibility push(default) void foo1() { __builtin_printf ("foo\n"); } +#pragma GCC visibility push(hidden) +void foo2() { __builtin_printf ("foo\n"); } #pragma GCC visibility pop -void foo2() { __builtin_printf ("foo\n"); } int main() { foo2(); return 0; }