[compiler-rt] ccd96b3 - [builtins][test] Fix divmodti4_test.c on Windows
Arthur Eubanks via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 23 19:09:14 PDT 2023
Author: Arthur Eubanks
Date: 2023-03-23T19:08:59-07:00
New Revision: ccd96b3e03e18653e909852bfef105fc10782acb
URL: https://github.com/llvm/llvm-project/commit/ccd96b3e03e18653e909852bfef105fc10782acb
DIFF: https://github.com/llvm/llvm-project/commit/ccd96b3e03e18653e909852bfef105fc10782acb.diff
LOG: [builtins][test] Fix divmodti4_test.c on Windows
By making the 64 bit integer literals unsigned. Otherwise some of them
are unexpectedly sign extended (and the compiler rightly diagnosed this
with warnings)
Initially added in D80506.
Reviewed By: MaskRay
Differential Revision: https://reviews.llvm.org/D146667
Added:
Modified:
compiler-rt/test/builtins/Unit/divmodti4_test.c
Removed:
################################################################################
diff --git a/compiler-rt/test/builtins/Unit/divmodti4_test.c b/compiler-rt/test/builtins/Unit/divmodti4_test.c
index 26b3c1609f18c..7b8d3faae2cdf 100644
--- a/compiler-rt/test/builtins/Unit/divmodti4_test.c
+++ b/compiler-rt/test/builtins/Unit/divmodti4_test.c
@@ -55,22 +55,22 @@ char assumption_1[sizeof(ti_int) == 2*sizeof(di_int)] = {0};
tu_int tests[][4] =
{
-{ (ti_int) 0, (ti_int) 1, (ti_int) 0, (ti_int) 0 },
-{ (ti_int) 0, (ti_int)-1, (ti_int) 0, (ti_int) 0 },
-{ (ti_int) 2, (ti_int) 1, (ti_int) 2, (ti_int) 0 },
-{ (ti_int) 2, (ti_int)-1, (ti_int)-2, (ti_int) 0 },
-{ (ti_int)-2, (ti_int) 1, (ti_int)-2, (ti_int) 0 },
-{ (ti_int)-2, (ti_int)-1, (ti_int) 2, (ti_int) 0 },
-{ (ti_int) 5, (ti_int) 3, (ti_int) 1, (ti_int) 2 },
-{ (ti_int) 5, (ti_int)-3, (ti_int)-1, (ti_int) 2 },
-{ (ti_int)-5, (ti_int) 3, (ti_int)-1, (ti_int)-2 },
-{ (ti_int)-5, (ti_int)-3, (ti_int) 1, (ti_int)-2 },
-{ (ti_int)0x8000000000000000LL << 64 | 0, (ti_int) 1, (ti_int)0x8000000000000000LL << 64 | 0, (ti_int)0x0LL },
-{ (ti_int)0x8000000000000000LL << 64 | 0, (ti_int)-1, (ti_int)0x8000000000000000LL << 64 | 0, (ti_int)0x0LL },
-{ (ti_int)0x8000000000000000LL << 64 | 0, (ti_int)-2, (ti_int)0x4000000000000000LL << 64 | 0, (ti_int)0x0LL },
-{ (ti_int)0x8000000000000000LL << 64 | 0, (ti_int) 2, (ti_int)0xC000000000000000LL << 64 | 0, (ti_int)0x0LL },
-{ (ti_int)0x8000000000000000LL << 64 | 0, (ti_int)-3, (ti_int)0x2AAAAAAAAAAAAAAALL << 64 | 0xAAAAAAAAAAAAAAAALL, (ti_int)-2 },
-{ (ti_int)0x8000000000000000LL << 64 | 0, (ti_int) 3, (ti_int)0xD555555555555555LL << 64 | 0x5555555555555556LL, (ti_int)-2 },
+{ (ti_int) 0, (ti_int) 1, (ti_int) 0, (ti_int) 0 },
+{ (ti_int) 0, (ti_int)-1, (ti_int) 0, (ti_int) 0 },
+{ (ti_int) 2, (ti_int) 1, (ti_int) 2, (ti_int) 0 },
+{ (ti_int) 2, (ti_int)-1, (ti_int)-2, (ti_int) 0 },
+{ (ti_int)-2, (ti_int) 1, (ti_int)-2, (ti_int) 0 },
+{ (ti_int)-2, (ti_int)-1, (ti_int) 2, (ti_int) 0 },
+{ (ti_int) 5, (ti_int) 3, (ti_int) 1, (ti_int) 2 },
+{ (ti_int) 5, (ti_int)-3, (ti_int)-1, (ti_int) 2 },
+{ (ti_int)-5, (ti_int) 3, (ti_int)-1, (ti_int)-2 },
+{ (ti_int)-5, (ti_int)-3, (ti_int) 1, (ti_int)-2 },
+{ (ti_int)0x8000000000000000ULL << 64 | 0, (ti_int) 1, (ti_int)0x8000000000000000ULL << 64 | 0, (ti_int)0x0LL },
+{ (ti_int)0x8000000000000000ULL << 64 | 0, (ti_int)-1, (ti_int)0x8000000000000000ULL << 64 | 0, (ti_int)0x0LL },
+{ (ti_int)0x8000000000000000ULL << 64 | 0, (ti_int)-2, (ti_int)0x4000000000000000ULL << 64 | 0, (ti_int)0x0LL },
+{ (ti_int)0x8000000000000000ULL << 64 | 0, (ti_int) 2, (ti_int)0xC000000000000000ULL << 64 | 0, (ti_int)0x0LL },
+{ (ti_int)0x8000000000000000ULL << 64 | 0, (ti_int)-3, (ti_int)0x2AAAAAAAAAAAAAAAULL << 64 | 0xAAAAAAAAAAAAAAAAULL, (ti_int)-2 },
+{ (ti_int)0x8000000000000000ULL << 64 | 0, (ti_int) 3, (ti_int)0xD555555555555555ULL << 64 | 0x5555555555555556ULL, (ti_int)-2 },
};
#endif
More information about the llvm-commits
mailing list