================
@@ -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);
+
+
+ if (Conf.CombinedIndexHook &&
----------------
luxufan wrote:
Without this change, some existing test cases would fail. This is because the
patch alters the position of the dead symbol analysis (moving it after
DeadRTTIElimIndex). So this change makes the dumped index file still include
information about dead symbols.
https://github.com/llvm/llvm-project/pull/126336
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
