On 26 September 2017 12:57:37 CEST, Segher Boessenkool
<[email protected]> wrote:
>> --- gcc/testsuite/gcc.target/powerpc/swps-p8-36.c (revision 0)
>> +++ gcc/testsuite/gcc.target/powerpc/swps-p8-36.c (working copy)
>> @@ -0,0 +1,31 @@
>> +/* This file's name was changed from swaps-p8-36.c so that the
>> + assembler search for "not swap" would not get a false
>> + positive on the name of the file. */
>
>Oh.
>
>> +/* { dg-final { scan-assembler-not "swap" } } */
>
>So what is this really testing for? xxswapd? But a) we never generate
>that, and b) you could use a better regex?
Yea this b) is generally what is recommended see also
https://gcc.gnu.org/ml/gcc-patches/2016-06/msg01358.html which reminds me that
the -fno-ident thing is still not upstream.
Or maybe we should have a -fno-file which suppresses .file like I proposed for
.ident. At least this would fix it once and for all.
>
>Or what else is it looking for? I bet b) holds anyway :-)
>
>Looks good except for those details.
>
>
>Segher
