From c3c031bb37d244ef3c95b08f81d66957da43ac4e Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Tue, 5 Jan 2010 17:50:58 +0000 Subject: Nick Lewycky pointed out that this code makes changes unconditionally. llvm-svn: 92739 --- llvm/lib/Transforms/Utils/Local.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'llvm/lib/Transforms/Utils') diff --git a/llvm/lib/Transforms/Utils/Local.cpp b/llvm/lib/Transforms/Utils/Local.cpp index 30fe8025135..1a7d27ad45c 100644 --- a/llvm/lib/Transforms/Utils/Local.cpp +++ b/llvm/lib/Transforms/Utils/Local.cpp @@ -328,7 +328,8 @@ llvm::RecursivelyDeleteDeadPHINode(PHINode *PN) { if (!PHIs.insert(cast(JP))) { // Break the cycle and delete the PHI and its operands. JP->replaceAllUsesWith(UndefValue::get(JP->getType())); - Changed |= RecursivelyDeleteTriviallyDeadInstructions(JP); + (void)RecursivelyDeleteTriviallyDeadInstructions(JP); + Changed = true; break; } return Changed; -- cgit v1.2.3