[llvm-commits] [llvm] r63597 - in /llvm/trunk: lib/Bitcode/Reader/BitcodeReader.cpp test/Bitcode/extractelement.ll

[Chris Lattner]

Author: lattner
Date: Mon Feb  2 20:11:28 2009
New Revision: 63597

URL: http://llvm.org/viewvc/llvm-project?rev=63597&view=rev
Log:
fix a bitcode reader bug where it can't handle extractelement correctly:
the index of the value being extracted is always an i32.  This fixes PR3465

Added:
    llvm/trunk/test/Bitcode/extractelement.ll
Modified:
    llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp

Modified: llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp?rev=63597&r1=63596&r2=63597&view=diff

==============================================================================
--- llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp (original)
+++ llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp Mon Feb  2 20:11:28 2009
@@ -915,8 +915,7 @@
         dyn_cast_or_null<VectorType>(getTypeByID(Record[0]));
       if (OpTy == 0) return Error("Invalid CE_EXTRACTELT record");
       Constant *Op0 = ValueList.getConstantFwdRef(Record[1], OpTy);
-      Constant *Op1 = ValueList.getConstantFwdRef(Record[2],
-                                                  OpTy->getElementType());
+      Constant *Op1 = ValueList.getConstantFwdRef(Record[2], Type::Int32Ty);
       V = ConstantExpr::getExtractElement(Op0, Op1);
       break;
     }

Added: llvm/trunk/test/Bitcode/extractelement.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Bitcode/extractelement.ll?rev=63597&view=auto

==============================================================================
--- llvm/trunk/test/Bitcode/extractelement.ll (added)
+++ llvm/trunk/test/Bitcode/extractelement.ll Mon Feb  2 20:11:28 2009
@@ -0,0 +1,8 @@
+; RUN: llvm-as < %s | opt -constprop | llvm-dis
+; PR3465
+
+define double @test() {
+  %tmp24 = extractelement <2 x double> bitcast (<1 x i128> < i128 85070591730234615870450834276742070272 > to <2 x double>), i32 0
+  ret double %tmp24
+}
+