diff options
| author | Kevin Enderby <enderby@apple.com> | 2012-10-18 21:49:18 +0000 |
|---|---|---|
| committer | Kevin Enderby <enderby@apple.com> | 2012-10-18 21:49:18 +0000 |
| commit | 88d12663abdac344f312b09edfe4934143436132 (patch) | |
| tree | 3c9f1cd7ca9617265e3112301c45f1fdafc8e84e /lib/Target/ARM/Disassembler/ARMDisassembler.cpp | |
| parent | 6220fb16c1d08534e770785f47717ee80baac517 (diff) | |
| download | external_llvm-88d12663abdac344f312b09edfe4934143436132.zip external_llvm-88d12663abdac344f312b09edfe4934143436132.tar.gz external_llvm-88d12663abdac344f312b09edfe4934143436132.tar.bz2 | |
Fix a bug where a 32-bit address with the high bit does not get symbolicated
because the value is incorrectly being signed extended when passed to
SymbolLookUp().
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166234 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/ARM/Disassembler/ARMDisassembler.cpp')
| -rw-r--r-- | lib/Target/ARM/Disassembler/ARMDisassembler.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/Target/ARM/Disassembler/ARMDisassembler.cpp b/lib/Target/ARM/Disassembler/ARMDisassembler.cpp index bf0dabb..d2b1cc3 100644 --- a/lib/Target/ARM/Disassembler/ARMDisassembler.cpp +++ b/lib/Target/ARM/Disassembler/ARMDisassembler.cpp @@ -525,8 +525,9 @@ static bool tryAddingSymbolicOperand(uint64_t Address, int32_t Value, else ReferenceType = LLVMDisassembler_ReferenceType_InOut_None; const char *ReferenceName; - const char *Name = SymbolLookUp(DisInfo, Value, &ReferenceType, Address, - &ReferenceName); + uint64_t SymbolValue = 0x00000000ffffffffULL & Value; + const char *Name = SymbolLookUp(DisInfo, SymbolValue, &ReferenceType, + Address, &ReferenceName); if (Name) { SymbolicOp.AddSymbol.Name = Name; SymbolicOp.AddSymbol.Present = true; |