From: Stepan Dyatkovskiy Date: Wed, 23 May 2012 08:18:26 +0000 (+0000) Subject: PR1255(case ranges) related changes in Local Transformations. X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=a2067fbe22930be8413584ae58c5ef78bd032190;p=oota-llvm.git PR1255(case ranges) related changes in Local Transformations. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@157315 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Transforms/Utils/Local.cpp b/lib/Transforms/Utils/Local.cpp index 4e1a00fa2d3..179f29cdafa 100644 --- a/lib/Transforms/Utils/Local.cpp +++ b/lib/Transforms/Utils/Local.cpp @@ -169,16 +169,20 @@ bool llvm::ConstantFoldTerminator(BasicBlock *BB, bool DeleteDeadConditions) { // Otherwise, we can fold this switch into a conditional branch // instruction if it has only one non-default destination. SwitchInst::CaseIt FirstCase = SI->case_begin(); - Value *Cond = Builder.CreateICmpEQ(SI->getCondition(), - FirstCase.getCaseValue(), "cond"); - - // Insert the new branch. - Builder.CreateCondBr(Cond, FirstCase.getCaseSuccessor(), - SI->getDefaultDest()); - - // Delete the old switch. - SI->eraseFromParent(); - return true; + ConstantRangesSet CRS = FirstCase.getCaseValueEx(); + if (CRS.getNumItems() == 1 && CRS.isSingleNumber(0)) { + Value *Cond = Builder.CreateICmpEQ(SI->getCondition(), + CRS.getItem(0).Low, "cond"); + + // Insert the new branch. + Builder.CreateCondBr(Cond, FirstCase.getCaseSuccessor(), + SI->getDefaultDest()); + + // Delete the old switch. + SI->eraseFromParent(); + return true; + + } } return false; }