[PATCH] D33615: Move summary dead stripping before regular LTO and record results in the combined summary

[Evgenii Stepanov via Phabricator via llvm-commits]

eugenis added a comment.

I've killed DeadSymbols.
ComputeCrossModuleImport may be called w/o computeDeadSymbols. In that case most of the symbols will be marked as dead (at that stage Live works like LiveRoot). I've added a flag in ModuleSummaryIndex to detect this situation.

This flag is not serialized. We  don't need it to be serialized, but that same is true for Live / LiveRoot. Actually, LiveRoot serialization was added in https://reviews.llvm.org/D23488 along with bitcode version bump, and, as far as I can see, it was unused from the start. I can kill the whole thing, and maybe even downgrade the version, if no one minds.


Repository:
  rL LLVM

https://reviews.llvm.org/D33615