[PATCH] D28076: [PM] Add support for building a default AA pipeline to the PassBuilder.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 23 01:47:15 PST 2016


davide accepted this revision.
davide added a reviewer: davide.
davide added a comment.
This revision is now accepted and ready to land.

LGTM. Side note, while reviewing it, I noticed there are still residuals of NoAA in-tree.

  llvm/InitializePasses.h:void initializeNoAAPass(PassRegistry&);



================
Comment at: lib/Passes/PassBuilder.cpp:1115-1119
+  // manager with our default one.
+  if (PipelineText == "default") {
+    AA = buildDefaultAAPipeline();
+    return true;
+  }
----------------
We need similar logic in lld (we already have a flag, `-lto-newpm-aa-pipeline=`. 


https://reviews.llvm.org/D28076





More information about the llvm-commits mailing list