diff options
author | Peter Collingbourne <peter@pcc.me.uk> | 2013-08-19 00:24:20 +0000 |
---|---|---|
committer | Peter Collingbourne <peter@pcc.me.uk> | 2013-08-19 00:24:20 +0000 |
commit | 353149ea2f8d4d3bf1ec82e90f80154c6959d56e (patch) | |
tree | ee70f1cfdc82e0684269d0f469f5f96587a76fa8 | |
parent | 57cf3500a3dec411500737b91e1a3970be488337 (diff) | |
download | external_llvm-353149ea2f8d4d3bf1ec82e90f80154c6959d56e.zip external_llvm-353149ea2f8d4d3bf1ec82e90f80154c6959d56e.tar.gz external_llvm-353149ea2f8d4d3bf1ec82e90f80154c6959d56e.tar.bz2 |
Remove SpecialCaseList::findCategory.
It turned out that I didn't need this for DFSan.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@188646 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | include/llvm/Transforms/Utils/SpecialCaseList.h | 16 | ||||
-rw-r--r-- | lib/Transforms/Utils/SpecialCaseList.cpp | 35 | ||||
-rw-r--r-- | unittests/Transforms/Utils/SpecialCaseList.cpp | 2 |
3 files changed, 0 insertions, 53 deletions
diff --git a/include/llvm/Transforms/Utils/SpecialCaseList.h b/include/llvm/Transforms/Utils/SpecialCaseList.h index bf95ec0..90eace1 100644 --- a/include/llvm/Transforms/Utils/SpecialCaseList.h +++ b/include/llvm/Transforms/Utils/SpecialCaseList.h @@ -83,20 +83,6 @@ class SpecialCaseList { /// omitted to search the empty category. bool isIn(const Module &M, const StringRef Category = StringRef()) const; - /// Returns whether either this function or its source file are listed in any - /// category. Category will contain the name of an arbitrary category in - /// which this function is listed. - bool findCategory(const Function &F, StringRef &Category) const; - - /// Returns whether this global, its type or its source file are listed in any - /// category. Category will contain the name of an arbitrary category in - /// which this global is listed. - bool findCategory(const GlobalVariable &G, StringRef &Category) const; - - /// Returns whether this module is listed in any category. Category will - /// contain the name of an arbitrary category in which this module is listed. - bool findCategory(const Module &M, StringRef &Category) const; - private: SpecialCaseList(SpecialCaseList const &) LLVM_DELETED_FUNCTION; SpecialCaseList &operator=(SpecialCaseList const &) LLVM_DELETED_FUNCTION; @@ -108,8 +94,6 @@ class SpecialCaseList { /// Parses just-constructed SpecialCaseList entries from a memory buffer. bool parse(const MemoryBuffer *MB, std::string &Error); - bool findCategory(const StringRef Section, const StringRef Query, - StringRef &Category) const; bool inSectionCategory(const StringRef Section, const StringRef Query, const StringRef Category) const; }; diff --git a/lib/Transforms/Utils/SpecialCaseList.cpp b/lib/Transforms/Utils/SpecialCaseList.cpp index 5400bcd..cf12bbf 100644 --- a/lib/Transforms/Utils/SpecialCaseList.cpp +++ b/lib/Transforms/Utils/SpecialCaseList.cpp @@ -169,12 +169,6 @@ SpecialCaseList::~SpecialCaseList() { } } -bool SpecialCaseList::findCategory(const Function &F, - StringRef &Category) const { - return findCategory(*F.getParent(), Category) || - findCategory("fun", F.getName(), Category); -} - bool SpecialCaseList::isIn(const Function& F, const StringRef Category) const { return isIn(*F.getParent(), Category) || inSectionCategory("fun", F.getName(), Category); @@ -191,13 +185,6 @@ static StringRef GetGVTypeString(const GlobalVariable &G) { return "<unknown type>"; } -bool SpecialCaseList::findCategory(const GlobalVariable &G, - StringRef &Category) const { - return findCategory(*G.getParent(), Category) || - findCategory("global", G.getName(), Category) || - findCategory("type", GetGVTypeString(G), Category); -} - bool SpecialCaseList::isIn(const GlobalVariable &G, const StringRef Category) const { return isIn(*G.getParent(), Category) || @@ -205,32 +192,10 @@ bool SpecialCaseList::isIn(const GlobalVariable &G, inSectionCategory("type", GetGVTypeString(G), Category); } -bool SpecialCaseList::findCategory(const Module &M, StringRef &Category) const { - return findCategory("src", M.getModuleIdentifier(), Category); -} - bool SpecialCaseList::isIn(const Module &M, const StringRef Category) const { return inSectionCategory("src", M.getModuleIdentifier(), Category); } -bool SpecialCaseList::findCategory(const StringRef Section, - const StringRef Query, - StringRef &Category) const { - StringMap<StringMap<Entry> >::const_iterator I = Entries.find(Section); - if (I == Entries.end()) return false; - - for (StringMap<Entry>::const_iterator II = I->second.begin(), - IE = I->second.end(); - II != IE; ++II) { - if (II->getValue().match(Query)) { - Category = II->first(); - return true; - } - } - - return false; -} - bool SpecialCaseList::inSectionCategory(const StringRef Section, const StringRef Query, const StringRef Category) const { diff --git a/unittests/Transforms/Utils/SpecialCaseList.cpp b/unittests/Transforms/Utils/SpecialCaseList.cpp index 42900b7..92a730d 100644 --- a/unittests/Transforms/Utils/SpecialCaseList.cpp +++ b/unittests/Transforms/Utils/SpecialCaseList.cpp @@ -94,8 +94,6 @@ TEST_F(SpecialCaseListTest, FunctionIsIn) { SCL.reset(makeSpecialCaseList("fun:foo=functional\n")); EXPECT_TRUE(SCL->isIn(*Foo, "functional")); StringRef Category; - EXPECT_TRUE(SCL->findCategory(*Foo, Category)); - EXPECT_EQ("functional", Category); EXPECT_FALSE(SCL->isIn(*Bar, "functional")); } |