[PATCH] D91585: [NPM] Move more O0 pass building into PassBuilder
Teresa Johnson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 17 12:41:07 PST 2020
tejohnson added a comment.
Thanks, generally this seems to be good cleanup. Question on one of the changes below though.
================
Comment at: llvm/lib/Passes/PassBuilder.cpp:2365
+ // Don't do anything for (thin)lto backend compiles at O0.
+ if (Matches[1] != "thinlto" && Matches[1] != "lto")
+ MPM.addPass(buildO0DefaultPipeline(L, Matches[1] != "default"));
----------------
This seems to change behavior. For one, previously we were only suppressing adding the PGO Instr passes for ThinLTO. Now this will suppress adding the coroutines passes and whatever else runRegisteredEPCallbacks was doing. Also, it's now doing the same for regular LTO but it didn't seem to do any special handling in that case before. Are these changes intended?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91585/new/
https://reviews.llvm.org/D91585
More information about the cfe-commits
mailing list