Re: [PATCH][ARM] Fix test fail with conflicting -mfloat-abi

[Sudakshina Das <sudi.das@arm.com>]

Hi Kyrill

On 19/01/18 18:00, Kyrill Tkachov wrote:
> 
> On 16/01/18 10:31, Sudakshina Das wrote:
>> Hi Christophe
>>
>> On 12/01/18 18:32, Christophe Lyon wrote:
>>> Le 12 janv. 2018 15:26, "Sudakshina Das" <sudi.das@arm.com> a écrit :
>>>
>>> Hi
>>>
>>> This patch fixes my earlier test case that fails for arm-none-eabi
>>> with explicit user option for -mfloat-abi which conflict with
>>> the test case options. I have added a guard to skip the test
>>> on those cases.
>>>
>>> @Christophe:
>>> Sorry about this. I think this should fix the test case.
>>> Can you please confirm if this works for you?
>>>
>>>
>>> Yes it does thanks
>>
>> Thanks for checking that. I have added one more directive for armv5t 
>> as well to avoid any conflicts for mcpu options.
>>
> 
> I agree with what Sudi said in 
> https://gcc.gnu.org/ml/gcc-patches/2018-01/msg01422.html
> I'd rather keep the test in the generic torture suite as long as we get 
> the directives right.
> 
> So this is ok for trunk (as the changes are arm-specific directives) 
> with one change below:
> 
> Thanks,
> Kyrill
> 
>> Sudi
>>
>>>
>>>
>>> Thanks
>>> Sudi
>>>
>>> gcc/testsuite/ChangeLog
>>>
>>> 2018-01-12  Sudakshina Das  <sudi.das@arm.com>
>>>
>>>          * gcc.c-torture/compile/pr82096.c: Add dg-skip-if
>>>          directive.
>>>
>>
> 
> 
> diff --git a/gcc/testsuite/gcc.c-torture/compile/pr82096.c 
> b/gcc/testsuite/gcc.c-torture/compile/pr82096.c
> index 9fed28c..35551f5 100644
> --- a/gcc/testsuite/gcc.c-torture/compile/pr82096.c
> +++ b/gcc/testsuite/gcc.c-torture/compile/pr82096.c
> @@ -1,3 +1,5 @@
> +/* { dg-require-effective-target arm_arch_v5t_ok } */
> 
> Please also guard this on { target arm*-*-* }
> That way this test will be run on other targets as well so that they can 
> benefit from it.
> 
> +/* { dg-skip-if "Do not combine float-abi values" { arm*-*-* } { 
> "-mfloat-abi=*" } { "-mfloat-abi=soft" } } */
>   /* { dg-additional-options "-march=armv5t -mthumb -mfloat-abi=soft" { 
> target arm*-*-* } } */
> 

Thanks committed with the change as r256941

Sudi

>