================
@@ -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));
----------------
ilovepi wrote:

Agreed. I'm not super happy w/ the boolean either. The only reason its required 
is to avoid asserting for the new use case, but that also implies that 
assertion doesn't always hold, so I'm not sure if its really a good idea to 
keep it.

https://github.com/llvm/llvm-project/pull/112788
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to