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

Thanks for splitting out a large part of the refactoring!



================
Comment at: include/llvm/IR/ModuleSummaryIndexYAML.h:132
+  unsigned Linkage;
+  bool NotEligibleToImport, Live;
   std::vector<uint64_t> TypeTests;
----------------
The addition of Linkage and NotEligibleToImport are unrelated to this change right? I feel you could commit them separately now.
(even the serialization of "Live" here isn't clear to me if it isn't something that could just be split out).


Repository:
  rL LLVM

https://reviews.llvm.org/D33615