diff options
author | Stepan Dyatkovskiy <stpworld@narod.ru> | 2012-05-12 10:48:17 +0000 |
---|---|---|
committer | Stepan Dyatkovskiy <stpworld@narod.ru> | 2012-05-12 10:48:17 +0000 |
commit | 1cce5bf8ef9ee3dc157ae5d8778f84a7a0d1d8b9 (patch) | |
tree | 378e29cf4f36c588be5cd059a60e6752c3338ae3 /lib/VMCore | |
parent | b7454fd9df0b477e3daf2fce6e1d5e1b241562df (diff) |
Recommited r156374 with critical fixes in BitcodeReader/Writer:
Ordinary patch for PR1255.
Added new case-ranges orientated methods for adding/removing cases in SwitchInst. After this patch cases will internally representated as ConstantArray-s instead of ConstantInt, externally cases wrapped within the ConstantRangesSet object.
Old methods of SwitchInst are also works well, but marked as deprecated. So on this stage we have no side effects except that I added support for case ranges in BitcodeReader/Writer, of course test for Bitcode is also added. Old "switch" format is also supported.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@156704 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/VMCore')
-rw-r--r-- | lib/VMCore/Instructions.cpp | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/lib/VMCore/Instructions.cpp b/lib/VMCore/Instructions.cpp index 6c5db32876..819a090cc3 100644 --- a/lib/VMCore/Instructions.cpp +++ b/lib/VMCore/Instructions.cpp @@ -3169,6 +3169,13 @@ SwitchInst::~SwitchInst() { /// addCase - Add an entry to the switch instruction... /// void SwitchInst::addCase(ConstantInt *OnVal, BasicBlock *Dest) { + CRSBuilder CB; + CB.add(OnVal); + ConstantRangesSet CRS = CB.getCase(); + addCase(CRS, Dest); +} + +void SwitchInst::addCase(ConstantRangesSet& OnVal, BasicBlock *Dest) { unsigned NewCaseIdx = getNumCases(); unsigned OpNo = NumOperands; if (OpNo+2 > ReservedSpace) @@ -3177,7 +3184,7 @@ void SwitchInst::addCase(ConstantInt *OnVal, BasicBlock *Dest) { assert(OpNo+1 < ReservedSpace && "Growing didn't work!"); NumOperands = OpNo+2; CaseIt Case(this, NewCaseIdx); - Case.setValue(OnVal); + Case.setValueEx(OnVal); Case.setSuccessor(Dest); } |