diff options
| author | Reid Kleckner <reid@kleckner.net> | 2009-08-19 23:39:59 +0000 |
|---|---|---|
| committer | Reid Kleckner <reid@kleckner.net> | 2009-08-19 23:39:59 +0000 |
| commit | ce4eb39bb430f5a87e8addc9a6900fcd34fff437 (patch) | |
| tree | bc346283a39e11e53efa69da286e853fbb11a862 /lib/Target/X86/X86JITInfo.cpp | |
| parent | 5e9bb73de6293cb39a0d18033a08bfb1eee656b0 (diff) | |
| download | llvm-ce4eb39bb430f5a87e8addc9a6900fcd34fff437.tar.gz llvm-ce4eb39bb430f5a87e8addc9a6900fcd34fff437.tar.bz2 llvm-ce4eb39bb430f5a87e8addc9a6900fcd34fff437.tar.xz | |
Modify an assert to avoid what looks like a GCC 4.2.4 signed-ness bug.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@79494 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/X86/X86JITInfo.cpp')
| -rw-r--r-- | lib/Target/X86/X86JITInfo.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/Target/X86/X86JITInfo.cpp b/lib/Target/X86/X86JITInfo.cpp index 9536fe672c..dea34a5123 100644 --- a/lib/Target/X86/X86JITInfo.cpp +++ b/lib/Target/X86/X86JITInfo.cpp @@ -495,9 +495,11 @@ void X86JITInfo::emitFunctionStubAtAddr(const Function* F, void *Fn, void *Stub, // complains about casting a function pointer to a normal pointer. JCE.startGVStub(F, Stub, 5); JCE.emitByte(0xE9); -#if defined (X86_64_JIT) - assert(((((intptr_t)Fn-JCE.getCurrentPCValue()-5) << 32) >> 32) == - ((intptr_t)Fn-JCE.getCurrentPCValue()-5) +#if defined (X86_64_JIT) && !defined (NDEBUG) + // Yes, we need both of these casts, or some broken versions of GCC (4.2.4) + // get the signed-ness of the expression wrong. Go figure. + intptr_t Displacement = (intptr_t)Fn - (intptr_t)JCE.getCurrentPCValue() - 5; + assert(((Displacement << 32) >> 32) == Displacement && "PIC displacement does not fit in displacement field!"); #endif JCE.emitWordLE((intptr_t)Fn-JCE.getCurrentPCValue()-4); |