[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