Also, aren't we pretty much the only target to default to PIC? So if
the 'target' and 'nonpic' testing is failing when used in the context
of dg-warning, wouldn't that appear functional on the nonpic targets?
That is, since the nonpic test is to check for __PIC__, if that fails
in dg-warning, wouldn't it 'appear' functional on nonpic targets, no?
On Mon, Feb 9, 2015 at 3:51 PM, Jack Howarth <howarth.at....@gmail.com> wrote:
> This appears to be the first instance where 'target' and 'nonpic' have
> been used in a dg-warning.
>
> On Mon, Feb 9, 2015 at 2:40 PM, Dominique d'Humières <domi...@lps.ens.fr>
> wrote:
>>
>>> Le 9 févr. 2015 à 20:01, Dominique d'Humières <domi...@lps.ens.fr> a écrit :
>>>
>>>
>>>> Le 9 févr. 2015 à 19:10, Tom de Vries <tom_devr...@mentor.com> a écrit :
>>>>
>>>> On 09-02-15 18:23, Dominique Dhumieres wrote:
>>>>> Tom,
>>>>>
>>>>> After these changes I have the following regressions on
>>>>> x86_64-apple-darwin1[04]*:
>>>>>
>>>>> FAIL: gcc.dg/uninit-19.c (test for warnings, line 22)
>>>>> FAIL: gcc.dg/uninit-19.c (test for excess errors)
>>>>> FAIL: gcc.dg/graphite/scop-19.c scan-tree-dump-times graphite "number of
>>>>> SCoPs: 0" 1
>>>>>
>>>>> I can prepare and test a fix for darwin unless you beat me!
>>>>>
>>>>
>>>> Dominique,
>>>>
>>>> thanks for finding this.
>>>>
>>>> I don't understand why this breaks. Why is fpic supposed to be different
>>>> for darwin?
>>>
>>> I don’t either, but the tests succeeds on darwin with the following patch
>>>
>>> --- ../_clean/gcc/testsuite/gcc.dg/uninit-19.c 2015-02-09
>>> 11:38:40.000000000 +0100
>>> +++ gcc/testsuite/gcc.dg/uninit-19.c 2015-02-09 19:52:26.000000000 +0100
>>> @@ -22,5 +22,5 @@ fn2 ()
>>> fn1 (l, &d, &e, &g, &i, &h, &k, n); /* 22. */
>>> }
>>>
>>> -/* { dg-warning "may be used uninitialized" "" { target nonpic } 13 } */
>>> -/* { dg-warning "may be used uninitialized" "" { target { ! nonpic } } 22
>>> } */
>>> +/* { dg-warning "may be used uninitialized" "" { target { nonpic ||
>>> x86_64-apple-darwin1[0-9]* } } 13 } */
>>> +/* { dg-warning "may be used uninitialized" "" { target { ! { nonpic ||
>>> x86_64-apple-darwin1[0-9]* } } } 22 } */
>>> --- ../_clean/gcc/testsuite/gcc.dg/graphite/scop-19.c 2015-02-09
>>> 11:38:40.000000000 +0100
>>> +++ gcc/testsuite/gcc.dg/graphite/scop-19.c 2015-02-09 19:55:38.000000000
>>> +0100
>>> @@ -31,7 +31,7 @@ d_growable_string_append_buffer (struct
>>> if (need > dgs->alc)
>>> d_growable_string_resize (dgs, need);
>>> }
>>> -/* { dg-final { scan-tree-dump-times "number of SCoPs: 0" 2 "graphite" {
>>> target nonpic } } } */
>>> -/* { dg-final { scan-tree-dump-times "number of SCoPs: 0" 1 "graphite" {
>>> target { ! nonpic } } } } */
>>> +/* { dg-final { scan-tree-dump-times "number of SCoPs: 0" 2 "graphite" {
>>> target { nonpic || x86_64-apple-darwin1[0-9]* } } } } */
>>> +/* { dg-final { scan-tree-dump-times "number of SCoPs: 0" 1 "graphite" {
>>> target { ! { nonpic || x86_64-apple-darwin1[0-9]* } } } } } */
>>> /* { dg-final { cleanup-tree-dump "graphite" } } */
>>>
>>> Cheers,
>>>
>>> Dominique
>>>
>>>> Thanks,
>>>> - Tom
>>>
>>
