diff options
author | Tim Northover <Tim.Northover@arm.com> | 2013-02-27 16:43:09 +0000 |
---|---|---|
committer | Tim Northover <Tim.Northover@arm.com> | 2013-02-27 16:43:09 +0000 |
commit | d65dfd83421f4d26e6dc20476718d7d9b6ba3f3b (patch) | |
tree | ce3f890710ef9c52a5343e82f2b5cfed0123ded6 | |
parent | 2fe71f8f7e678174ba217d3d16525092805ad45b (diff) | |
download | llvm-d65dfd83421f4d26e6dc20476718d7d9b6ba3f3b.tar.gz llvm-d65dfd83421f4d26e6dc20476718d7d9b6ba3f3b.tar.bz2 llvm-d65dfd83421f4d26e6dc20476718d7d9b6ba3f3b.tar.xz |
ARM: permit full range of valid ADR immediates.
This fixes an issue where trying to assemlbe valid ADR instructions would cause
LLVM to hit a failed assertion.
Patch by Keith Walker.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@176189 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Target/ARM/MCTargetDesc/ARMMCCodeEmitter.cpp | 17 | ||||
-rw-r--r-- | test/MC/ARM/basic-arm-instructions.s | 4 |
2 files changed, 19 insertions, 2 deletions
diff --git a/lib/Target/ARM/MCTargetDesc/ARMMCCodeEmitter.cpp b/lib/Target/ARM/MCTargetDesc/ARMMCCodeEmitter.cpp index 09e15afd3c..7a59a7dd50 100644 --- a/lib/Target/ARM/MCTargetDesc/ARMMCCodeEmitter.cpp +++ b/lib/Target/ARM/MCTargetDesc/ARMMCCodeEmitter.cpp @@ -655,15 +655,28 @@ getAdrLabelOpValue(const MCInst &MI, unsigned OpIdx, int32_t offset = MO.getImm(); uint32_t Val = 0x2000; + int SoImmVal; if (offset == INT32_MIN) { Val = 0x1000; - offset = 0; + SoImmVal = 0; } else if (offset < 0) { Val = 0x1000; offset *= -1; + SoImmVal = ARM_AM::getSOImmVal(offset); + if(SoImmVal == -1) { + Val = 0x2000; + offset *= -1; + SoImmVal = ARM_AM::getSOImmVal(offset); + } + } else { + SoImmVal = ARM_AM::getSOImmVal(offset); + if(SoImmVal == -1) { + Val = 0x1000; + offset *= -1; + SoImmVal = ARM_AM::getSOImmVal(offset); + } } - int SoImmVal = ARM_AM::getSOImmVal(offset); assert(SoImmVal != -1 && "Not a valid so_imm value!"); Val |= SoImmVal; diff --git a/test/MC/ARM/basic-arm-instructions.s b/test/MC/ARM/basic-arm-instructions.s index 45ea278aac..560a0d633c 100644 --- a/test/MC/ARM/basic-arm-instructions.s +++ b/test/MC/ARM/basic-arm-instructions.s @@ -143,11 +143,15 @@ Lforward: adr r1, #-0x0 adr r1, #-0x12000000 + adr r1, #-0x80000001 adr r1, #0x12000000 + adr r1, #0x80000001 @ CHECK: adr r1, #-0 @ encoding: [0x00,0x10,0x4f,0xe2] @ CHECK: adr r1, #-301989888 @ encoding: [0x12,0x14,0x4f,0xe2] +@ CHECK: adr r1, #2147483647 @ encoding: [0x06,0x11,0x4f,0xe2] @ CHECK: adr r1, #301989888 @ encoding: [0x12,0x14,0x8f,0xe2] +@ CHECK: adr r1, #-2147483647 @ encoding: [0x06,0x11,0x8f,0xe2] @------------------------------------------------------------------------------ |