From 89d46b0f095cf7f536c6ad40b4541b73d259445f Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Thu, 23 May 2002 16:52:34 +0000 Subject: [PATCH] Fix bug: test/Regression/Transforms/ADCE/2002-05-23-ZeroArgPHITest.ll Which contains a description of why this is neccesary. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2733 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/VMCore/BasicBlock.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/VMCore/BasicBlock.cpp b/lib/VMCore/BasicBlock.cpp index 9a6301eb016..512ddb59df9 100644 --- a/lib/VMCore/BasicBlock.cpp +++ b/lib/VMCore/BasicBlock.cpp @@ -8,6 +8,7 @@ #include "llvm/iTerminators.h" #include "llvm/Type.h" #include "llvm/Support/CFG.h" +#include "llvm/Constant.h" #include "llvm/iPHINode.h" #include "llvm/CodeGen/MachineInstr.h" @@ -131,7 +132,9 @@ void BasicBlock::removePredecessor(BasicBlock *Pred) { // If the PHI _HAD_ two uses, replace PHI node with its now *single* value if (max_idx == 2) PN->replaceAllUsesWith(PN->getOperand(0)); - delete getInstList().remove(begin()); // Remove the PHI node + else // Otherwise there are no incoming values/edges, replace with dummy + PN->replaceAllUsesWith(Constant::getNullValue(PN->getType())); + delete getInstList().remove(begin()); // Remove the PHI node } } else { // Okay, now we know that we need to remove predecessor #pred_idx from all -- 2.34.1