[llvm-branch-commits] [clang] 0176e87 - [Clang] Fix JIT test on 32-bit systems

[Tobias Hieta via llvm-branch-commits]

Author: Sam James
Date: 2023-09-11T09:07:51+02:00
New Revision: 0176e8729ea4a2cff1ec6689c7620a9f37ce9904

URL: https://github.com/llvm/llvm-project/commit/0176e8729ea4a2cff1ec6689c7620a9f37ce9904
DIFF: https://github.com/llvm/llvm-project/commit/0176e8729ea4a2cff1ec6689c7620a9f37ce9904.diff

LOG: [Clang] Fix JIT test on 32-bit systems

As reported by mgorny at https://reviews.llvm.org/D159115#4638037, the
unsigned long long cast fails on 32-bit systems at least with GCC.

It looks like a pointer provenance/aliasing issue rather than a bug in GCC.

Acked by Vassil Vassilev on the original revision.

(cherry picked from commit 3403686b72507e3fdbcd69f21fb9235ffa0ca169)

Added: 
    

Modified: 
    clang/unittests/Interpreter/InterpreterTest.cpp

Removed: 
    


################################################################################
diff  --git a/clang/unittests/Interpreter/InterpreterTest.cpp b/clang/unittests/Interpreter/InterpreterTest.cpp
index abb8e6377aabddb..5f2911e9a7adad3 100644
--- a/clang/unittests/Interpreter/InterpreterTest.cpp
+++ b/clang/unittests/Interpreter/InterpreterTest.cpp
@@ -244,7 +244,7 @@ TEST(IncrementalProcessing, FindMangledNameSymbol) {
 
   // FIXME: Re-enable when we investigate the way we handle dllimports on Win.
 #ifndef _WIN32
-  EXPECT_EQ((unsigned long long)&printf, Addr->getValue());
+  EXPECT_EQ((uintptr_t)&printf, Addr->getValue());
 #endif // _WIN32
 }