diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2009-04-16 12:34:53 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2009-04-16 12:34:53 +0000 |
| commit | 5e42177a0f2a5bda77ffb00bae1caa7e6eb373f2 (patch) | |
| tree | dd004f2760ec82c55ed0b5b619467fdc90c6d522 | |
| parent | 092f0ccd9c994da4e9bd5bb402a4d9776d1befd3 (diff) | |
| download | bcm5719-llvm-5e42177a0f2a5bda77ffb00bae1caa7e6eb373f2.tar.gz bcm5719-llvm-5e42177a0f2a5bda77ffb00bae1caa7e6eb373f2.zip | |
fix PR3995. A scale must be 1, 2, 4 or 8.
llvm-svn: 69284
| -rw-r--r-- | llvm/lib/Target/X86/X86ISelDAGToDAG.cpp | 2 | ||||
| -rw-r--r-- | llvm/test/CodeGen/X86/2009-04-scale.ll | 23 |
2 files changed, 24 insertions, 1 deletions
diff --git a/llvm/lib/Target/X86/X86ISelDAGToDAG.cpp b/llvm/lib/Target/X86/X86ISelDAGToDAG.cpp index 18a574fc5ae..4b698cec1a5 100644 --- a/llvm/lib/Target/X86/X86ISelDAGToDAG.cpp +++ b/llvm/lib/Target/X86/X86ISelDAGToDAG.cpp @@ -1041,7 +1041,7 @@ bool X86DAGToDAGISel::MatchAddress(SDValue N, X86ISelAddressMode &AM, // a scaled index. if (Shift.getOpcode() == ISD::SRL && Shift.hasOneUse()) { unsigned ScaleLog = 8 - C1->getZExtValue(); - if (ScaleLog > 0 && ScaleLog < 64 && + if (ScaleLog > 0 && ScaleLog < 4 && C2->getZExtValue() == (UINT64_C(0xff) << ScaleLog)) { SDValue Eight = CurDAG->getConstant(8, MVT::i8); SDValue Mask = CurDAG->getConstant(0xff, N.getValueType()); diff --git a/llvm/test/CodeGen/X86/2009-04-scale.ll b/llvm/test/CodeGen/X86/2009-04-scale.ll new file mode 100644 index 00000000000..0766dc79e02 --- /dev/null +++ b/llvm/test/CodeGen/X86/2009-04-scale.ll @@ -0,0 +1,23 @@ +; RUN: llvm-as < %s | llc -march=x86 -mtriple=i386-unknown-linux-gnu +; PR3995 + + %struct.vtable = type { i32 (...)** } + %struct.array = type { %struct.impl, [256 x %struct.pair], [256 x %struct.pair], [256 x %struct.pair], [256 x %struct.pair], [256 x %struct.pair], [256 x %struct.pair] } + %struct.impl = type { %struct.vtable, i8, %struct.impl*, i32, i32, i64, i64 } + %struct.pair = type { i64, i64 } + +define void @test() { +entry: + %0 = load i32* null, align 4 ; <i32> [#uses=1] + %1 = lshr i32 %0, 8 ; <i32> [#uses=1] + %2 = and i32 %1, 255 ; <i32> [#uses=1] + %3 = getelementptr %struct.array* null, i32 0, i32 3 ; <[256 x %struct.pair]*> [#uses=1] + %4 = getelementptr [256 x %struct.pair]* %3, i32 0, i32 %2 ; <%struct.pair*> [#uses=1] + %5 = getelementptr %struct.pair* %4, i32 0, i32 1 ; <i64*> [#uses=1] + %6 = load i64* %5, align 4 ; <i64> [#uses=1] + %7 = xor i64 0, %6 ; <i64> [#uses=1] + %8 = xor i64 %7, 0 ; <i64> [#uses=1] + %9 = xor i64 %8, 0 ; <i64> [#uses=1] + store i64 %9, i64* null, align 8 + unreachable +} |
