[PATCH] D17477: [Polly] [RFC] Allow the client of DependenceInfo to obtain dependences at different granularities.

Michael Kruse via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 05:24:24 PST 2016


Meinersbur added a subscriber: Meinersbur.

================
Comment at: include/polly/DependenceInfo.h:58
@@ +57,3 @@
+
+    NUM_ANALYSIS_LEVELS
+  };
----------------
For consistent naming, please refer to the [[ http://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly | LLVM style guide ]]

================
Comment at: lib/Analysis/DependenceInfo.cpp:797
@@ -775,3 +796,3 @@
 void DependenceInfo::getAnalysisUsage(AnalysisUsage &AU) const {
   ScopPass::getAnalysisUsage(AU);
 }
----------------
ScopPass::getAnalysisUsage(AU) 
adds 
addRequired<ScopInfo>();
what you'd need with this change is
addRequiredTransitive<ScopInfo>();

To avoid transitive requirements, I suggest per-granularity DependenceInfo passes:
DependenceInfo(Dependences::AnalysisLevel) {} 


Repository:
  rL LLVM

http://reviews.llvm.org/D17477





More information about the llvm-commits mailing list