Re: [testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Tom de Vries
On 12/14/2017 03:18 PM, Rainer Orth wrote: Hi Jakub, On Thu, Dec 14, 2017 at 03:09:02PM +0100, Tom de Vries wrote: On 12/14/2017 02:47 PM, Jakub Jelinek wrote: On Thu, Dec 14, 2017 at 02:40:12PM +0100, Tom de Vries wrote: --- a/gcc/testsuite/gcc.dg/visibility-22.c +++ b/gcc/testsuite/gcc.dg/

Re: [testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Rainer Orth
Hi Jakub, > On Thu, Dec 14, 2017 at 03:09:02PM +0100, Tom de Vries wrote: >> On 12/14/2017 02:47 PM, Jakub Jelinek wrote: >> > On Thu, Dec 14, 2017 at 02:40:12PM +0100, Tom de Vries wrote: >> > > --- a/gcc/testsuite/gcc.dg/visibility-22.c >> > > +++ b/gcc/testsuite/gcc.dg/visibility-22.c >> > > @@

Re: [testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Rainer Orth
Hi Tom, > this patch introduces an effective target weak_undefined, and uses it in > test-case visibility-22.c. > > I've tested this on trunk for x86_64, and the test still runs and passes. > > I've tested this on an internal branch for the gcn target, where it makes > the test unsupported (instea

Re: [testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Jakub Jelinek
On Thu, Dec 14, 2017 at 03:09:02PM +0100, Tom de Vries wrote: > On 12/14/2017 02:47 PM, Jakub Jelinek wrote: > > On Thu, Dec 14, 2017 at 02:40:12PM +0100, Tom de Vries wrote: > > > --- a/gcc/testsuite/gcc.dg/visibility-22.c > > > +++ b/gcc/testsuite/gcc.dg/visibility-22.c > > > @@ -1,6 +1,7 @@ > >

Re: [testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Tom de Vries
On 12/14/2017 02:47 PM, Jakub Jelinek wrote: On Thu, Dec 14, 2017 at 02:40:12PM +0100, Tom de Vries wrote: --- a/gcc/testsuite/gcc.dg/visibility-22.c +++ b/gcc/testsuite/gcc.dg/visibility-22.c @@ -1,6 +1,7 @@ /* PR target/32219 */ /* { dg-do run } */ /* { dg-require-visibility "" } */ +/*

Re: [testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Jakub Jelinek
On Thu, Dec 14, 2017 at 02:40:12PM +0100, Tom de Vries wrote: > --- a/gcc/testsuite/gcc.dg/visibility-22.c > +++ b/gcc/testsuite/gcc.dg/visibility-22.c > @@ -1,6 +1,7 @@ > /* PR target/32219 */ > /* { dg-do run } */ > /* { dg-require-visibility "" } */ > +/* { dg-require-effective-target weak_un