[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#769259, @eugenis wrote:

> 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.


I'm not sure I understand this? LiveRoot was added to track things like globals marked as "llvm.used", how would this work without serialization from the compile phase to the thinlink?


Repository:
  rL LLVM

https://reviews.llvm.org/D33615