ignore 'invoke new' in isInstructionTriviallyDead, since most callers are not ready to handle invokes. instcombine will take care of this.

nunoplopes · nunoplopes · commit b37ef71ce157 · 2012-06-29T17:37:07.000Z
llvm-svn: 159440
diff --git a/llvm/lib/Transforms/Utils/Local.cpp b/llvm/lib/Transforms/Utils/Local.cpp
@@ -265,7 +265,8 @@ bool llvm::isInstructionTriviallyDead(Instruction *I) {
       return isa<UndefValue>(II->getArgOperand(1));
   }
 
-  if (isAllocLikeFn(I)) return true;
+  if (isAllocLikeFn(I))
+    return isa<CallInst>(I); // do not mess around with invoke here
 
   if (CallInst *CI = isFreeCall(I))
     if (Constant *C = dyn_cast<Constant>(CI->getArgOperand(0)))

Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,8 @@ bool llvm::isInstructionTriviallyDead(Instruction *I) {`
`265`	`265`	`return isa<UndefValue>(II->getArgOperand(1));`
`266`	`266`	`}`
`267`	`267`
`268`		`- if (isAllocLikeFn(I)) return true;`
	`268`	`+ if (isAllocLikeFn(I))`
	`269`	`+ return isa<CallInst>(I); // do not mess around with invoke here`
`269`	`270`
`270`	`271`	`if (CallInst *CI = isFreeCall(I))`
`271`	`272`	`if (Constant *C = dyn_cast<Constant>(CI->getArgOperand(0)))`