[PATCH] D146200: [PassBuilder] Support O0 in default pipelines

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea accepted this revision.
asbirlea added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:1554
 ModulePassManager
 PassBuilder::buildLTOPreLinkDefaultPipeline(OptimizationLevel Level) {
   // FIXME: We should use a customized pre-link pipeline!
----------------
nikic wrote:
> asbirlea wrote:
> > I'm missing why you're not calling `buildO0DefaultPipeline` for O0 here too.
> buildPerModuleDefaultPipeline() will call buildO0DefaultPipeline() in this case.
Ah, right! Thanks!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146200/new/

https://reviews.llvm.org/D146200