diff options
| author | Nick Lewycky <nicholas@mxc.ca> | 2008-08-08 06:34:07 +0000 |
|---|---|---|
| committer | Nick Lewycky <nicholas@mxc.ca> | 2008-08-08 06:34:07 +0000 |
| commit | d6227385fc499a8b8bc23766735f1d809d194378 (patch) | |
| tree | ffe502d87a5771aa7e3bf3a21861d20588cffa21 | |
| parent | 65ad452536526fe9a80023abc3703a7cc7987858 (diff) | |
Don't crash printing the asm for a ConstantExpr PtrToInt just because the int
is narrower than the pointer. This testcase emits:
.byte (((17) - 16) & 255)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@54517 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/CodeGen/AsmPrinter.cpp | 9 | ||||
| -rw-r--r-- | test/CodeGen/Generic/2008-08-07-PtrToInt-SmallerInt.ll | 7 |
2 files changed, 12 insertions, 4 deletions
diff --git a/lib/CodeGen/AsmPrinter.cpp b/lib/CodeGen/AsmPrinter.cpp index 366b8b398f..1ee71bdc72 100644 --- a/lib/CodeGen/AsmPrinter.cpp +++ b/lib/CodeGen/AsmPrinter.cpp @@ -821,12 +821,13 @@ void AsmPrinter::EmitConstantValueOnly(const Constant *CV) { // We can emit the pointer value into this slot if the slot is an // integer slot greater or equal to the size of the pointer. - if (Ty->isInteger() && - TD->getABITypeSize(Ty) >= TD->getABITypeSize(Op->getType())) + if (TD->getABITypeSize(Ty) >= TD->getABITypeSize(Op->getType())) return EmitConstantValueOnly(Op); - - assert(0 && "FIXME: Don't yet support this kind of constant cast expr"); + + O << "(("; EmitConstantValueOnly(Op); + APInt ptrMask = APInt::getAllOnesValue(TD->getABITypeSizeInBits(Ty)); + O << ") & " << ptrMask.toStringUnsigned() << ')'; break; } case Instruction::Add: diff --git a/test/CodeGen/Generic/2008-08-07-PtrToInt-SmallerInt.ll b/test/CodeGen/Generic/2008-08-07-PtrToInt-SmallerInt.ll new file mode 100644 index 0000000000..a60d101fee --- /dev/null +++ b/test/CodeGen/Generic/2008-08-07-PtrToInt-SmallerInt.ll @@ -0,0 +1,7 @@ +; RUN: llvm-as < %s | llc +; PR2603 +target datalayout = "e-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:32:64-f32:32:32-f64:32:64-v64:64:64-v128:128:128-a0:0:64-f80:32:32" +target triple = "i386-pc-linux-gnu" + %struct.A = type { i8 } + %struct.B = type { i8, [1 x i8] } +@Foo = constant %struct.A { i8 ptrtoint (i8* getelementptr ([1 x i8]* inttoptr (i32 17 to [1 x i8]*), i32 0, i32 -16) to i8) } ; <%struct.A*> [#uses=0] |