[PATCH] D17010: Add an "addUsedAAAnalyses" helper function

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 8 16:49:41 PST 2016


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

LGTM, minor suggestion below.

Not a huge fan of the name, but no better ideas, and it doesn't matter much.


================
Comment at: lib/Analysis/AliasAnalysis.cpp:568-572
@@ +567,7 @@
+void llvm::addUsedAAAnalyses(AnalysisUsage &AU) {
+  AU.addUsedIfAvailable<ScopedNoAliasAAWrapperPass>();
+  AU.addUsedIfAvailable<TypeBasedAAWrapperPass>();
+  AU.addUsedIfAvailable<objcarc::ObjCARCAAWrapperPass>();
+  AU.addUsedIfAvailable<GlobalsAAWrapperPass>();
+  AU.addUsedIfAvailable<CFLAAWrapperPass>();
+}
----------------
Add a comment here and above indicating how these should be kept in sync? Minor.


http://reviews.llvm.org/D17010





More information about the llvm-commits mailing list