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

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

In https://reviews.llvm.org/D33615#768262, @eugenis wrote:

> To recap: this needs to be in combined summary, because that's the only way to pass information to LowerTypeTests in regularLTO.


Well you can also pass it as a set just like it is done elsewhere with `DeadSymbols`. 
However, if we go the route of "caching" this analysis result in the summaries, then we should likely kill `DeadSymbols` because `ComputeCrossModuleImport` should be able to use the information from the summary directly.


Repository:
  rL LLVM

https://reviews.llvm.org/D33615