This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: patch to fix PR65648
- From: Yvan Roux <yvan dot roux at linaro dot org>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Vladimir Makarov <vmakarov at redhat dot com>, Terry Guo <Terry dot Guo at arm dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 13 Apr 2015 16:10:52 +0200
- Subject: Re: patch to fix PR65648
- Authentication-results: sourceware.org; auth=none
- References: <5523F167 dot 3010606 at redhat dot com> <20150407155143 dot GK19273 at tucnak dot redhat dot com> <CAD57uCcf_5zj=_P_ASkF34ZMVBusVbN7bEz_9aP-ir3qx7Ocdg at mail dot gmail dot com> <20150407193339 dot GN19273 at tucnak dot redhat dot com> <CAD57uCeGZJpxUTUm3BP38mStFNs0q2Fyn-rpGtSBVYsftHB3yQ at mail dot gmail dot com> <CAD57uCcwLCJttGUoLDJcfJF16di+w5F9Hp2B78fywnpaMD32UQ at mail dot gmail dot com> <552BC7D1 dot 8010106 at arm dot com>
On 13 April 2015 at 15:42, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
>
> On 09/04/15 12:10, Yvan Roux wrote:
>>
>> diff --git a/gcc/testsuite/gcc.target/arm/pr65648.c
>> b/gcc/testsuite/gcc.target/arm/pr65648.c
>> new file mode 100644
>> index 0000000..e075546
>> --- /dev/null
>> +++ b/gcc/testsuite/gcc.target/arm/pr65648.c
>> @@ -0,0 +1,9 @@
>> +/* { dg-do run } */
>> +/* { dg-skip-if "avoid conflicting multilib options" { *-*-* } {
>> "-march=*" } { "-march=armv6" } } */
>> +/* { dg-skip-if "avoid conflicting multilib options" { *-*-* } { "-marm"
>> } { "" } } */
>> +/* { dg-skip-if "avoid conflicting multilib options" { *-*-*eabihf } {
>> "*" } { "" } } */
>> +/* { dg-options "-mthumb -Os -mfloat-abi=soft" } */
>> +/* { dg-add-options arm_arch_v6 } */
>> +
>> +#include "../../gcc.c-torture/execute/pr65648.c"
>> +
>
> Hi Yvan,
>
> These are always tough to get right.
> How about:
> /* { dg-skip-if "avoid conflicting multilib options" { *-*-*eabihf } { "*" }
> { "" } } */
> /* { dg-options "-Os -mthumb -mfloat-abi=soft" } */
> /* { dg-add-options arm_arch_v6 } */
> /* { dg-require-effective-target arm_arch_v6_ok } */
> ?
>
> I think the dg-skip-if will avoid the error when testing
> arm-none-linux-gnueabihf:
> "error: ./pr65648.exe uses VFP register arguments, /tmp/ccXpRQ41.o does
> not"
>
> The dg-require-effective-target should remove the need for the first
> dg-skip-if in your options.
> I don't think it's worth skipping the test when the user explicitly asks for
> -marm. It won't test the
> behaviour of the bug but then again, the user overrode the options, so
> presumably knows best.
Yes it looks better to me, the -marm skipping in my patch was an
artifact of testing it for armv6-m
> Is there any case where this fails?
none that I can think of, Is it ok to commit after I've re-tested it
(maybe once 5.1 is released) ?
Yvan