Re: [PATCH] Fix PR68621

2016-03-02 Thread Richard Biener
On Tue, Mar 1, 2016 at 5:48 PM, Kumar, Venkataramanan wrote: > Hi Richard, > > As discussed in PR, tried to adjust the test case by initializing array, but > looks like for building with -fpic it needs visibility to be set a hidden. > The below patch does that. Making the array 'static' would al

Re: [PATCH] Fix PR68621

2016-03-01 Thread Jeff Law
On 03/01/2016 09:48 AM, Kumar, Venkataramanan wrote: Hi Richard, As discussed in PR, tried to adjust the test case by initializing array, but looks like for building with -fpic it needs visibility to be set a hidden. The below patch does that. Ok for trunk ? diff --git a/gcc/testsuite/Change

RE: [PATCH] Fix PR68621

2016-03-01 Thread Kumar, Venkataramanan
ent: Tuesday, March 01, 2016 10:23 PM > To: Kumar, Venkataramanan > Cc: Richard Beiner (richard.guent...@gmail.com); gcc-patches@gcc.gnu.org; > hjl.to...@gmail.com > Subject: Re: [PATCH] Fix PR68621 > > On Tue, Mar 01, 2016 at 04:48:40PM +, Kumar, Venkataramanan wrote: > >

Re: [PATCH] Fix PR68621

2016-03-01 Thread Marek Polacek
On Tue, Mar 01, 2016 at 04:48:40PM +, Kumar, Venkataramanan wrote: > /* { dg-do compile } */ > -/* { dg-options "-Ofast -fdump-tree-ifcvt-details -fno-common > -ftree-loop-if-convert-stores" } */ > +/* { dg-options "-Ofast -fdump-tree-ifcvt-details > -ftree-loop-if-convert-stores" } */ > >

[PATCH] Fix PR68621

2016-03-01 Thread Kumar, Venkataramanan
Hi Richard, As discussed in PR, tried to adjust the test case by initializing array, but looks like for building with -fpic it needs visibility to be set a hidden. The below patch does that. Ok for trunk ? diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 82e538e..f6bcb07 10