diff options
author | Stepan Dyatkovskiy <stpworld@narod.ru> | 2012-05-23 08:18:26 +0000 |
---|---|---|
committer | Stepan Dyatkovskiy <stpworld@narod.ru> | 2012-05-23 08:18:26 +0000 |
commit | a2067fbe22930be8413584ae58c5ef78bd032190 (patch) | |
tree | 53e4cd89152767cc19b2a413c108c7c5f5075865 | |
parent | 88097819fca78aa9f5d1607d0e6271a096c5bd07 (diff) | |
download | external_llvm-a2067fbe22930be8413584ae58c5ef78bd032190.zip external_llvm-a2067fbe22930be8413584ae58c5ef78bd032190.tar.gz external_llvm-a2067fbe22930be8413584ae58c5ef78bd032190.tar.bz2 |
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
-rw-r--r-- | lib/Transforms/Utils/Local.cpp | 24 |
1 files changed, 14 insertions, 10 deletions
diff --git a/lib/Transforms/Utils/Local.cpp b/lib/Transforms/Utils/Local.cpp index 4e1a00f..179f29c 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; } |