On Wed, Jul 31, 2024 at 01:32:06PM +0200, Filip Kastl wrote:
> Thanks for the feedback! Here is a second version of the patch. I've tested
> this version with
>
> make check RUNTESTFLAGS="i386.exp=gcc.target/i386/switch-exp-transform-3.c
> --target_board='unix{-m32}'"
>
> and
>
> make check RUNTESTFLAGS="i386.exp=gcc.target/i386/switch-exp-transform-3.c"
You can just use
make check RUNTESTFLAGS="--target_board=unix\{-m32,-m64\}
i386.exp=switch-exp-transform-3.c"
> testsuite: Adjust switch-exp-transform-3.c for 32bit
>
> 32bit x86 CPUs won't natively support the FFS operation on a 64 bit
> type. Therefore, I'm setting the long long int part of the
> switch-exp-transform-3.c test to only execute with 64bit targets.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.target/i386/switch-exp-transform-3.c: Set the long long
> int test to only execute with 64bit targets.
There should be just tab, not tab + 2 spaces before int test.
Ok with that nit changed.
Jakub
