From 61ddab6d4c61eae27b52105661f860cdd2e87192 Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Wed, 22 Jul 2015 22:32:34 +0000 Subject: [PATCH] [GMR] Continue my quest to remove linked datastructures from GMR, NFC. This replaces the next-to-last std::map with a DenseMap. While DenseMap doesn't yet make tons of sense (there are 32 bytes or so in the value type), my next change will reduce the value type to a single pointer -- we only need a pointer and 3 bits, and that is exactly what we can have. llvm-svn: 242956 --- llvm/lib/Analysis/IPA/GlobalsModRef.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/llvm/lib/Analysis/IPA/GlobalsModRef.cpp b/llvm/lib/Analysis/IPA/GlobalsModRef.cpp index cfefa0b..3e4b00e 100644 --- a/llvm/lib/Analysis/IPA/GlobalsModRef.cpp +++ b/llvm/lib/Analysis/IPA/GlobalsModRef.cpp @@ -101,7 +101,7 @@ class GlobalsModRef : public ModulePass, public AliasAnalysis { /// FunctionInfo - For each function, keep track of what globals are /// modified or read. - std::map FunctionInfo; + DenseMap FunctionInfo; /// Handle to clear this analysis on deletion of values. struct DeletionCallbackHandle final : CallbackVH { @@ -227,8 +227,7 @@ private: /// getFunctionInfo - Return the function info for the function, or null if /// we don't have anything useful to say about it. FunctionRecord *getFunctionInfo(const Function *F) { - std::map::iterator I = - FunctionInfo.find(F); + auto I = FunctionInfo.find(F); if (I != FunctionInfo.end()) return &I->second; return nullptr; -- 2.7.4