Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

2013-05-27 Thread Patrick Marlier
Hi, This is just a memo about why the testcase failed: When it tries to compile, we get this: gcc/testsuite/g++.dg/tm/pr47746.C:20:14: error: unsafe function call ‘void Building::load(InputStream*)’ within ‘transaction_safe’ function load(stream); Indeed, with PIC, the 'load' method can be ove

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

2013-05-27 Thread Alexander Ivchenko
Oh, sorry. Missed the last msg from Mike in that thread. 2013/5/27 Alexander Ivchenko : > *ping* > > Alexander > > 2013/4/18 Patrick Marlier : >> Hi Alexander, >> >> On Thu, Apr 18, 2013 at 12:49 PM, Alexander Ivchenko >> wrote: >>> I'm trying it on linux/x86_64 on trunk. Testing just by adding

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

2013-05-27 Thread Alexander Ivchenko
*ping* Alexander 2013/4/18 Patrick Marlier : > Hi Alexander, > > On Thu, Apr 18, 2013 at 12:49 PM, Alexander Ivchenko > wrote: >> I'm trying it on linux/x86_64 on trunk. Testing just by adding -fpic >> to the dg-options: >> >> --- a/gcc/testsuite/g++.dg/tm/pr47746.C >> +++ b/gcc/testsuite/g++.d

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

2013-05-24 Thread Mike Stump
On May 24, 2013, at 12:02 AM, Alexander Ivchenko wrote: > *ping* > > 2013/4/11 Alexander Ivchenko : >> Hi, >> >> The same motivation as for: >> http://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00786.html >> >> "Since -fpic option is turned on by default in Android we have certain test >> fails. The rea

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

2013-05-24 Thread Alexander Ivchenko
*ping* 2013/4/11 Alexander Ivchenko : > Hi, > > The same motivation as for: > http://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00786.html > > "Since -fpic option is turned on by default in Android we have certain test > fails. The reason for that is that those tests rely on the > availability of functions

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

2013-04-18 Thread Patrick Marlier
Hi Alexander, On Thu, Apr 18, 2013 at 12:49 PM, Alexander Ivchenko wrote: > I'm trying it on linux/x86_64 on trunk. Testing just by adding -fpic > to the dg-options: > > --- a/gcc/testsuite/g++.dg/tm/pr47746.C > +++ b/gcc/testsuite/g++.dg/tm/pr47746.C > @@ -1,5 +1,5 @@ > // { dg-do compile } > -

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

2013-04-18 Thread Alexander Ivchenko
Hi Patrick, I'm trying it on linux/x86_64 on trunk. Testing just by adding -fpic to the dg-options: --- a/gcc/testsuite/g++.dg/tm/pr47746.C +++ b/gcc/testsuite/g++.dg/tm/pr47746.C @@ -1,5 +1,5 @@ // { dg-do compile } -// { dg-options "-fgnu-tm" } +// { dg-options "-fgnu-tm -fpic" } Here is the

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

2013-04-18 Thread Patrick Marlier
Hi Alexander, On Thu, Apr 11, 2013 at 11:37 AM, Alexander Ivchenko wrote: > The same motivation as for: > http://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00786.html > > "Since -fpic option is turned on by default in Android we have certain test > fails. The reason for that is that those tests rely on th