]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
testsuite: remove -fwrapv from signbit-5.c
authorTamar Christina <tamar.christina@arm.com>
Thu, 5 Sep 2024 09:36:02 +0000 (10:36 +0100)
committerTamar Christina <tamar.christina@arm.com>
Thu, 5 Sep 2024 09:36:02 +0000 (10:36 +0100)
The meaning of the testcase was changed by passing it -fwrapv.  The reason for
the test failures on some platform was because the test was testing some
implementation defined behavior wrt INT_MIN in generic code.

Instead of using -fwrapv this just removes the border case from the test so
all the values now have a defined semantic.  It still relies on the handling of
shifting a negative value right, but that wasn't changed with -fwrapv anyway.

The -fwrapv case is being handled already by other testcases.

gcc/testsuite/ChangeLog:

* gcc.dg/signbit-5.c: Remove -fwrapv and change INT_MIN to INT_MIN+1.

gcc/testsuite/gcc.dg/signbit-5.c

index 57e29e3ca63efbb0ae34010f8a275ebc247b14ff..2601582ed4ea24830edabd5e44f8cd9f56c8d9c0 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do run } */
-/* { dg-options "-O3 -fwrapv" } */
+/* { dg-options "-O3" } */
 
 /* This test does not work when the truth type does not match vector type.  */
 /* { dg-additional-options "-march=armv8-a" { target aarch64_sve } } */
@@ -42,8 +42,8 @@ int main ()
   TYPE a[N];
   TYPE b[N];
 
-  a[0] = INT_MIN;
-  b[0] = INT_MIN;
+  a[0] = INT_MIN+1;
+  b[0] = INT_MIN+1;
 
   for (int i = 1; i < N; ++i)
     {