"Thomas Preud'homme" <thomas.preudho...@arm.com> writes:
>> From: Joseph Myers [mailto:jos...@codesourcery.com]
>>
>> > + if { [is-effective-target bswap]
>> > + && ![istarget x86_64-*-*] } {
>>
>> That x86_64-*-* test is wrong. x86_64-*-* and i?86-*-* should always be
>> handled the same (if you then want to distinguish 32-bit and 64-bit
>> multilibs, you check the appropriate effective-target there, depending on
>> whether the condition is one on the ABI or which register size is being
>> used, which affects how x32 should be counted).
>
> Indeed, it's a mistake. I?86 should be in there two. Please find attached an
> updated patch.
> diff --git a/gcc/testsuite/gcc.dg/optimize-bswapdi-1.c
> b/gcc/testsuite/gcc.dg/optimize-bswapdi-1.c
> index 7d557f3..a9c3443 100644
> --- a/gcc/testsuite/gcc.dg/optimize-bswapdi-1.c
> +++ b/gcc/testsuite/gcc.dg/optimize-bswapdi-1.c
> @@ -1,6 +1,6 @@
> -/* { dg-do compile { target arm*-*-* alpha*-*-* ia64*-*-* x86_64-*-*
> s390x-*-* powerpc*-*-* rs6000-*-* } } */
> +/* { dg-do compile { target *-*-* } } */
Just omit the { target *-*-* } completely, also a few more times.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
