Part of r159527. Splitted into series of patches and gone with fixed PR13256:

IntegersSubsetMapping
  - Replaced type of Items field from std::list with std::map. In neares future I'll test it with DenseMap and do the correspond replacement
    if possible.



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@159659 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Stepan Dyatkovskiy
2012-07-03 13:46:45 +00:00
parent 181e0bdafa
commit 6a59073735
3 changed files with 18 additions and 42 deletions

View File

@ -238,13 +238,14 @@ unsigned LowerSwitch::Clusterify(CaseVector& Cases, SwitchInst *SI) {
size_t numCmps = 0;
for (IntegersSubsetToBB::RangeIterator i = TheClusterifier.begin(),
e = TheClusterifier.end(); i != e; ++i, ++numCmps) {
IntegersSubsetToBB::Cluster &C = *i;
const IntegersSubsetToBB::RangeTy &R = i->first;
BasicBlock *S = i->second;
// FIXME: Currently work with ConstantInt based numbers.
// Changing it to APInt based is a pretty heavy for this commit.
Cases.push_back(CaseRange(C.first.getLow().toConstantInt(),
C.first.getHigh().toConstantInt(), C.second));
if (C.first.isSingleNumber())
Cases.push_back(CaseRange(R.getLow().toConstantInt(),
R.getHigh().toConstantInt(), S));
if (R.isSingleNumber())
// A range counts double, since it requires two compares.
++numCmps;
}