the type field for a store is the type of the pointer, not the value.

With this fix I can round trip treeaadd, only losing calling conv info.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36706 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Bitcode/Reader/BitcodeReader.cpp b/lib/Bitcode/Reader/BitcodeReader.cpp
index 1ab9f54d5bf2bd1cc09555a762be2d75250f3fb9..8396d442fc5fbf9cab1b0a5570c5acd880aa3609 100644 (file)
--- a/lib/Bitcode/Reader/BitcodeReader.cpp
+++ b/lib/Bitcode/Reader/BitcodeReader.cpp
@@ -1351,9 +1351,10 @@ bool BitcodeReader::ParseFunctionBody(Function *F) {
     case bitc::FUNC_CODE_INST_STORE: { // STORE:[ptrty,val,ptr, align, vol]
       if (Record.size() < 5)
         return Error("Invalid LOAD record");
-      const Type *OpTy = getTypeByID(Record[0]);
-      Value *Op = getFnValueByID(Record[1], OpTy);
-      Value *Ptr = getFnValueByID(Record[2], PointerType::get(OpTy));
+      const PointerType *OpTy = 
+        dyn_cast_or_null<PointerType>(getTypeByID(Record[0]));
+      Value *Op = getFnValueByID(Record[1], OpTy ? OpTy->getElementType() : 0);
+      Value *Ptr = getFnValueByID(Record[2], OpTy);
       if (!OpTy || !Op || !Ptr)
         return Error("Invalid STORE record");
       I = new StoreInst(Op, Ptr, (1 << Record[3]) >> 1, Record[4]);