[PATCH] D78512: [llvm] Factor out inlining pipeline as a module pipeline.

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin marked 2 inline comments as done.
mtrofin added inline comments.


================
Comment at: llvm/lib/Passes/PassRegistry.def:85
 MODULE_PASS("sample-profile", SampleProfileLoaderPass())
+MODULE_PASS("scc-oz-module-inliner",
+  buildInlinerPipeline(OptimizationLevel::Oz, ThinLTOPhase::None, DebugLogging))
----------------
davidxl wrote:
> Why oz? Perhaps barebone-scc-inliner ?
because I'm passing OptimizationLevel::Oz. If we want other OptimizationLevels, we'd add other names, e.g. MODULE_PASS("scc-o3-module-inliner", etc. We start with -oz because I plan on leveraging this for the ML patches.

...if there's a way to pass the optimization level otherwise, happy to do that instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78512