Hi,

Finally got my hands on that: the attached patch adds the
"target nonpic" for those tests that require the avaiability of
functions defined in them.

OK for trunk?

thanks,
Alexander


2013/1/10 Richard Biener <richard.guent...@gmail.com>:
> On Thu, Jan 10, 2013 at 2:50 PM, Alexander Ivchenko <aivch...@gmail.com> 
> wrote:
>> Hi,
>>
>> It all begun with discussion here
>>  http://gcc.gnu.org/ml/gcc/2012-11/msg00205.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 conservatively assumes that they are 
>> AVAIL_OVERWRITABLE.
>>
>> The attached patch adding -fno-pic option for tests that fail because of 
>> that.
>
> I think this should be a dg-requires nopic instead.  Otherwise testing
> with -fPIC/-fno-pic will not show expected differences.
>
> Richard.
>
>> 2013-01-10  Alexander Ivchenko  <alexander.ivche...@intel.com>
>>
>>        * g++.dg/ipa/ivinline-1.C: Add -fno-pic option.
>>        * g++.dg/ipa/ivinline-2.C: Likewise.
>>        * g++.dg/ipa/ivinline-3.C: Likewise.
>>        * g++.dg/ipa/ivinline-4.C: Likewise.
>>        * g++.dg/ipa/ivinline-5.C: Likewise.
>>        * g++.dg/ipa/ivinline-7.C: Likewise.
>>        * g++.dg/ipa/ivinline-8.C: Likewise.
>>        * g++.dg/ipa/ivinline-9.C: Likewise.
>>        * g++.dg/cpp0x/noexcept03.C: Likewise.
>>        * gcc.dg/const-1.c: Likewise.
>>        * gcc.dg/ipa/pure-const-1.c: Likewise.
>>        * gcc.dg/noreturn-8.c: Likewise.
>>        * gcc.dg/tree-ssa/ipa-split-5.c: Likewise.
>>        * gcc.dg/tree-ssa/loadpre6.c: Likewise.
>>        * gcc.c-torture/execute/pr33992.c: Likewise.
>>        * gcc.c-torture/execute/pr33992.x: New file.
>>
>> ok for mainline?
>>
>> thanks,
>> Alexander

Attachment: adding_fno_pic_02.patch
Description: Binary data

Reply via email to