fix a bug exposed by moving SRoA earlier which caused a crash building kc++
authorChris Lattner <sabre@nondot.org>
Mon, 2 Nov 2009 04:37:17 +0000 (04:37 +0000)
committerChris Lattner <sabre@nondot.org>
Mon, 2 Nov 2009 04:37:17 +0000 (04:37 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@85786 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Utils/PromoteMemoryToRegister.cpp
test/Transforms/Mem2Reg/crash.ll

index b70276e255e963f2cf688666591643c33db362f7..de6ad1dde580bbe35c081df145dfe474222d2ea1 100644 (file)
@@ -749,7 +749,12 @@ void PromoteMem2Reg::RewriteSingleStoreAlloca(AllocaInst *AI,
     }
     
     // Otherwise, we *can* safely rewrite this load.
-    LI->replaceAllUsesWith(OnlyStore->getOperand(0));
+    Value *ReplVal = OnlyStore->getOperand(0);
+    // If the replacement value is the load, this must occur in unreachable
+    // code.
+    if (ReplVal == LI)
+      ReplVal = UndefValue::get(LI->getType());
+    LI->replaceAllUsesWith(ReplVal);
     if (AST && isa<PointerType>(LI->getType()))
       AST->deleteValue(LI);
     LI->eraseFromParent();
index ce795aaaca4ba88835afc2ceac2cf3245ee47eef..655549f79402cb8735d205403451523c9143376f 100644 (file)
@@ -1,12 +1,12 @@
 ; RUN: opt < %s -mem2reg -S
 ; PR5023
 
-declare i32 @bar()
+declare i32 @test1f()
 
-define i32 @foo() {
+define i32 @test1() {
 entry:
   %whichFlag = alloca i32
-  %A = invoke i32 @bar()
+  %A = invoke i32 @test1f()
           to label %invcont2 unwind label %lpad86
 
 invcont2:
@@ -22,3 +22,20 @@ lpad86:
   
 }
 
+
+
+define i32 @test2() {
+entry:
+  %whichFlag = alloca i32
+  br label %bb15
+
+bb15:
+  %B = load i32* %whichFlag
+  ret i32 %B
+
+invcont2:
+  %C = load i32* %whichFlag
+  store i32 %C, i32* %whichFlag
+  br label %bb15
+}
+