[llvm-branch-commits] [llvm][fatlto] Drop any CFI related instrumentation after emitting bitcode (PR #112788)
via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Thu Oct 17 16:30:05 PDT 2024
================
@@ -1629,6 +1629,10 @@ PassBuilder::buildFatLTODefaultPipeline(OptimizationLevel Level, bool ThinLTO,
MPM.addPass(buildLTOPreLinkDefaultPipeline(Level));
MPM.addPass(EmbedBitcodePass(ThinLTO, EmitSummary));
+ // If we're doing FatLTO w/ CFI enabled, we don't want the type tests in the
+ // object file.
+ MPM.addPass(LowerTypeTestsPass(nullptr, nullptr, true, true));
----------------
pcc wrote:
Oh I see, you have a dependent change that adds the argument.
https://github.com/llvm/llvm-project/commit/68a064b39b86d69c02de09cda2f06f4b87c2e69a
I'm not really a big fan of adding another bool argument like this because it makes the callers less clear, can you do something else like make the existing argument into an enum?
https://github.com/llvm/llvm-project/pull/112788
More information about the llvm-branch-commits
mailing list