[clang] [llvm] [ThinLTO] Support dead RTTI data elimination under -fno-split-lto-unit (PR #126336)

Teresa Johnson via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 12 11:01:51 PST 2025


================
@@ -1920,6 +1924,18 @@ Error LTO::runThinLTO(AddStreamFn AddStream, FileCache Cache,
       ThinLTO.CombinedIndex, WholeProgramVisibilityEnabledInLTO,
       DynamicExportSymbols, VisibleToRegularObjSymbols);
 
+  Triple TT(getTargetTriple());
+  DeadRTTIElimIndex(ThinLTO.CombinedIndex, TT).run();
+
+  if (!ThinLTO.CombinedIndex.withGlobalValueDeadStripping())
+    computeDeadSymbolsWithConstProp(ThinLTO.CombinedIndex, GUIDPreservedSymbols,
+                                    IsPrevailing, Conf.OptLevel > 0);
----------------
teresajohnson wrote:

Can we just move up and use the isPrevailing lambda defined a bit further below here (line 1946)?

https://github.com/llvm/llvm-project/pull/126336


More information about the cfe-commits mailing list