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

2013-05-27 Thread Alexander Ivchenko
*ping* Alexander 2013/4/18 Patrick Marlier patrick.marl...@gmail.com: Hi Alexander, On Thu, Apr 18, 2013 at 12:49 PM, Alexander Ivchenko aivch...@gmail.com wrote: I'm trying it on linux/x86_64 on trunk. Testing just by adding -fpic to the dg-options: ---

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 aivch...@gmail.com: *ping* Alexander 2013/4/18 Patrick Marlier patrick.marl...@gmail.com: Hi Alexander, On Thu, Apr 18, 2013 at 12:49 PM, Alexander Ivchenko aivch...@gmail.com wrote: I'm trying it on

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

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

2013-05-24 Thread Alexander Ivchenko
*ping* 2013/4/11 Alexander Ivchenko aivch...@gmail.com: 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

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 aivch...@gmail.com wrote: *ping* 2013/4/11 Alexander Ivchenko aivch...@gmail.com: 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

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 aivch...@gmail.com 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

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 18, 2013 at 12:49 PM, Alexander Ivchenko aivch...@gmail.com 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

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

2013-04-11 Thread 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, defined in them and with -fpic compiler