On 2/26/25 06:03, Alex Bennée wrote:
> We allow things like:
>
> tests/tcg/i386/test-i386-bmi2.c:124:35: warning: shifting a negative signed value is undefined [-Wshift-negative-value]
> assert(result == (mask & ~(-1 << 30)));
>
> in the main code, so allow it for the test.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/tcg/i386/Makefile.target | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/tcg/i386/Makefile.target b/tests/tcg/i386/Makefile.target
> index bbe2c44b2a..f1df40411b 100644
> --- a/tests/tcg/i386/Makefile.target
> +++ b/tests/tcg/i386/Makefile.target
> @@ -22,7 +22,7 @@ run-test-i386-sse-exceptions: QEMU_OPTS += -cpu max
> test-i386-pcmpistri: CFLAGS += -msse4.2
> run-test-i386-pcmpistri: QEMU_OPTS += -cpu max
>
> -test-i386-bmi2: CFLAGS=-O2
> +test-i386-bmi2: CFLAGS=-O2 -fwrapv
> run-test-i386-bmi2: QEMU_OPTS += -cpu max
>
> test-i386-adcox: CFLAGS=-O2
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~