On Wed 2024-07-31 13:34:28, Jakub Jelinek wrote:
> 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
>
Ok, removed the 2 spaces and pushed the patch.
Thanks,
Filip Kastl
