[llvm] r306355 - [CFLAA] Move FunctionHandle to llvm::cflaa.

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 19:43:00 PDT 2017


Author: davide
Date: Mon Jun 26 19:43:00 2017
New Revision: 306355

URL: http://llvm.org/viewvc/llvm-project?rev=306355&view=rev
Log:
[CFLAA] Move FunctionHandle to llvm::cflaa.

Also, while here, remove an unneeded `using namespace llvm`.
Thanks to George for the suggestion.

Modified:
    llvm/trunk/include/llvm/Analysis/CFLAliasAnalysisUtils.h
    llvm/trunk/include/llvm/Analysis/CFLAndersAliasAnalysis.h
    llvm/trunk/include/llvm/Analysis/CFLSteensAliasAnalysis.h

Modified: llvm/trunk/include/llvm/Analysis/CFLAliasAnalysisUtils.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/CFLAliasAnalysisUtils.h?rev=306355&r1=306354&r2=306355&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/CFLAliasAnalysisUtils.h (original)
+++ llvm/trunk/include/llvm/Analysis/CFLAliasAnalysisUtils.h Mon Jun 26 19:43:00 2017
@@ -18,7 +18,8 @@
 #include "llvm/IR/Function.h"
 #include "llvm/IR/ValueHandle.h"
 
-using namespace llvm;
+namespace llvm {
+namespace cflaa {
 
 template <typename AAResult> struct FunctionHandle final : public CallbackVH {
   FunctionHandle(Function *Fn, AAResult *Result)
@@ -41,8 +42,6 @@ private:
   }
 };
 
-namespace llvm {
-namespace cflaa {
 static inline const Function *parentFunctionOfValue(const Value *Val) {
   if (auto *Inst = dyn_cast<Instruction>(Val)) {
     auto *Bb = Inst->getParent();

Modified: llvm/trunk/include/llvm/Analysis/CFLAndersAliasAnalysis.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/CFLAndersAliasAnalysis.h?rev=306355&r1=306354&r2=306355&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/CFLAndersAliasAnalysis.h (original)
+++ llvm/trunk/include/llvm/Analysis/CFLAndersAliasAnalysis.h Mon Jun 26 19:43:00 2017
@@ -76,7 +76,7 @@ private:
   /// that simply has empty sets.
   DenseMap<const Function *, Optional<FunctionInfo>> Cache;
 
-  std::forward_list<FunctionHandle<CFLAndersAAResult>> Handles;
+  std::forward_list<cflaa::FunctionHandle<CFLAndersAAResult>> Handles;
 };
 
 /// Analysis pass providing a never-invalidated alias analysis result.

Modified: llvm/trunk/include/llvm/Analysis/CFLSteensAliasAnalysis.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/CFLSteensAliasAnalysis.h?rev=306355&r1=306354&r2=306355&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/CFLSteensAliasAnalysis.h (original)
+++ llvm/trunk/include/llvm/Analysis/CFLSteensAliasAnalysis.h Mon Jun 26 19:43:00 2017
@@ -94,7 +94,7 @@ private:
   /// have any kind of recursion, it is discernable from a function
   /// that simply has empty sets.
   DenseMap<Function *, Optional<FunctionInfo>> Cache;
-  std::forward_list<FunctionHandle<CFLSteensAAResult>> Handles;
+  std::forward_list<cflaa::FunctionHandle<CFLSteensAAResult>> Handles;
 
   FunctionInfo buildSetsFrom(Function *F);
 };




More information about the llvm-commits mailing list