[PATCH] D12357: [PM] Add full support for alias analyses to the new pass manager, and port aa-eval to the new pass manager to test this support.

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 27 12:17:32 PDT 2015


hfinkel added inline comments.

================
Comment at: lib/Passes/PassBuilder.cpp:229
@@ +228,3 @@
+  // the way the parameter to \c registerFunctionAnalyses does.
+  AAManager AAM;
+
----------------
chandlerc wrote:
> hfinkel wrote:
> > I don't understand why this helps. Where is the variable referenced?
> The CREATE_PASS bit for the AAManager expands to "std::move(AAM)" below, because that's what registerFunctionAnalyses wants so that you can customize the AA manager that ends up in your analysis manager.
Ah, okay. Please update the comment to explain that (that it is referenced by the CREATE_PASS macro).



http://reviews.llvm.org/D12357





More information about the llvm-commits mailing list