IntegersSubsetMapping: removed exclude operation, it will replaced with more universal "diff" operation in next commit.

Changes was separated onto two commits for better readability.

llvm-svn: 159200
This commit is contained in:
Stepan Dyatkovskiy 2012-06-26 11:41:47 +00:00
parent b4e01abdb4
commit 883850c4d2
2 changed files with 0 additions and 226 deletions

View File

@ -66,12 +66,6 @@ protected:
} }
}; };
struct ClusterLefterThan {
bool operator()(const Cluster &C, const RangeTy &R) {
return C.first.getHigh() < R.getLow();
}
};
CaseItems Items; CaseItems Items;
bool Sorted; bool Sorted;
@ -103,40 +97,6 @@ protected:
} }
} }
void exclude(CaseItemIt &beginIt, RangeTy &R) {
std::list<CaseItemIt> ToBeErased;
CaseItemIt endIt = Items.end();
CaseItemIt It =
std::lower_bound(beginIt, Items.end(), R, ClusterLefterThan());
if (It == endIt)
return;
if (It->first.getLow() < R.getLow())
Items.insert(It, std::make_pair(
RangeTy(It->first.getLow(), R.getLow()-1),
It->second));
do
ToBeErased.push_back(It++);
while (It != endIt && It->first.getLow() <= R.getHigh());
beginIt = It;
CaseItemIt &LastRemoved = *(--ToBeErased.end());
if (LastRemoved->first.getHigh() > R.getHigh())
beginIt = Items.insert(LastRemoved, std::make_pair(
RangeTy(R.getHigh() + 1, LastRemoved->first.getHigh()),
LastRemoved->second
));
for (typename std::list<CaseItemIt>::iterator i = ToBeErased.begin(),
e = ToBeErased.end(); i != e; ++i)
Items.erase(*i);
}
public: public:
// Don't public CaseItems itself. Don't allow edit the Items directly. // Don't public CaseItems itself. Don't allow edit the Items directly.
@ -230,18 +190,6 @@ public:
/// Removes items from set. /// Removes items from set.
void removeItem(RangeIterator i) { Items.erase(i); } void removeItem(RangeIterator i) { Items.erase(i); }
// Excludes RHS subset from current mapping. RHS should consists of non
// overlapped ranges only and sorted from left to the right.
// method will have unpredictional behaviour in another case.
void exclude(IntegersSubsetTy &RHS) {
CaseItemIt startIt = begin();
for (unsigned i = 0, e = RHS.getNumItems();
i != e && startIt != end(); ++i) {
RangeTy R = RHS.getItem(i);
exclude(startIt, R);
}
}
/// Builds the finalized case objects. /// Builds the finalized case objects.
void getCases(Cases& TheCases) { void getCases(Cases& TheCases) {
CRSMap TheCRSMap; CRSMap TheCRSMap;

View File

@ -178,178 +178,4 @@ namespace {
EXPECT_EQ(CaseIt->second.getItem(0), Range(Int(i * 10), Int(i * 10 + 9))); EXPECT_EQ(CaseIt->second.getItem(0), Range(Int(i * 10), Int(i * 10 + 9)));
} }
} }
TEST(IntegersSubsetTest, ExcludeTest) {
std::vector<Range> Ranges;
Ranges.reserve(3);
Mapping TheMapping;
// Test case
// { {0, 4}, {7, 10} {13, 17} }
// sub
// { {3, 14} }
// =
// { {0, 2}, {15, 17} }
Ranges.push_back(Range(Int(0), Int(4)));
Ranges.push_back(Range(Int(7), Int(10)));
Ranges.push_back(Range(Int(13), Int(17)));
Subset TheSubset(Ranges);
TheMapping.add(TheSubset);
Ranges.clear();
Ranges.push_back(Range(Int(3), Int(14)));
TheSubset = Subset(Ranges);
TheMapping.exclude(TheSubset);
TheSubset = TheMapping.getCase();
EXPECT_EQ(TheSubset.getNumItems(), 2ULL);
EXPECT_EQ(TheSubset.getItem(0), Range(Int(0), Int(2)));
EXPECT_EQ(TheSubset.getItem(1), Range(Int(15), Int(17)));
// Test case
// { {0, 4}, {7, 10} {13, 17} }
// sub
// { {0, 4}, {13, 17} }
// =
// { {7, 10 }
Ranges.clear();
Ranges.push_back(Range(Int(0), Int(4)));
Ranges.push_back(Range(Int(7), Int(10)));
Ranges.push_back(Range(Int(13), Int(17)));
TheSubset = Subset(Ranges);
TheMapping.clear();
TheMapping.add(TheSubset);
Ranges.clear();
Ranges.push_back(Range(Int(0), Int(4)));
Ranges.push_back(Range(Int(13), Int(17)));
TheSubset = Subset(Ranges);
TheMapping.exclude(TheSubset);
TheSubset = TheMapping.getCase();
EXPECT_EQ(TheSubset.getNumItems(), 1ULL);
EXPECT_EQ(TheSubset.getItem(0), Range(Int(7), Int(10)));
// Test case
// { {0, 17} }
// sub
// { {1, 5}, {10, 12}, {15, 16} }
// =
// { {0}, {6, 9}, {13, 14}, {17} }
Ranges.clear();
Ranges.push_back(Range(Int(0), Int(17)));
TheSubset = Subset(Ranges);
TheMapping.clear();
TheMapping.add(TheSubset);
Ranges.clear();
Ranges.push_back(Range(Int(1), Int(5)));
Ranges.push_back(Range(Int(10), Int(12)));
Ranges.push_back(Range(Int(15), Int(16)));
TheSubset = Subset(Ranges);
TheMapping.exclude(TheSubset);
TheSubset = TheMapping.getCase();
EXPECT_EQ(TheSubset.getNumItems(), 4ULL);
EXPECT_EQ(TheSubset.getItem(0), Range(Int(0)));
EXPECT_EQ(TheSubset.getItem(1), Range(Int(6), Int(9)));
EXPECT_EQ(TheSubset.getItem(2), Range(Int(13), Int(14)));
EXPECT_EQ(TheSubset.getItem(3), Range(Int(17)));
// Test case
// { {2, 4} }
// sub
// { {0, 5} }
// =
// { empty }
Ranges.clear();
Ranges.push_back(Range(Int(2), Int(4)));
TheSubset = Subset(Ranges);
TheMapping.clear();
TheMapping.add(TheSubset);
Ranges.clear();
Ranges.push_back(Range(Int(0), Int(5)));
TheSubset = Subset(Ranges);
TheMapping.exclude(TheSubset);
EXPECT_TRUE(TheMapping.empty());
// Test case
// { {2, 4} }
// sub
// { {7, 8} }
// =
// { {2, 4} }
Ranges.clear();
Ranges.push_back(Range(Int(2), Int(4)));
TheSubset = Subset(Ranges);
TheMapping.clear();
TheMapping.add(TheSubset);
Ranges.clear();
Ranges.push_back(Range(Int(7), Int(8)));
TheSubset = Subset(Ranges);
TheMapping.exclude(TheSubset);
TheSubset = TheMapping.getCase();
EXPECT_EQ(TheSubset.getNumItems(), 1ULL);
EXPECT_EQ(TheSubset.getItem(0), Range(Int(2), Int(4)));
// Test case
// { {3, 7} }
// sub
// { {1, 4} }
// =
// { {5, 7} }
Ranges.clear();
Ranges.push_back(Range(Int(3), Int(7)));
TheSubset = Subset(Ranges);
TheMapping.clear();
TheMapping.add(TheSubset);
Ranges.clear();
Ranges.push_back(Range(Int(1), Int(4)));
TheSubset = Subset(Ranges);
TheMapping.exclude(TheSubset);
TheSubset = TheMapping.getCase();
EXPECT_EQ(TheSubset.getNumItems(), 1ULL);
EXPECT_EQ(TheSubset.getItem(0), Range(Int(5), Int(7)));
}
} }