[PATCH] D12114: [LPM] Teach the legacy pass manager to support *using* an analysis without *requiring* it.
Chandler Carruth via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 18 14:47:48 PDT 2015
chandlerc added inline comments.
================
Comment at: include/llvm/PassAnalysisSupport.h:95
@@ +94,3 @@
+ template<class PassClass>
+ AnalysisUsage &addUsed() {
+ Used.push_back(&PassClass::ID);
----------------
hfinkel wrote:
> Should we call this addUsedIfAvailable, or similar, to make it clearer what this does (and, specifically, how it differs from addRequired)?
>
Seems a small gain, but if you like. I don't really care. You (or others) give me a spelling.
http://reviews.llvm.org/D12114
More information about the llvm-commits
mailing list