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

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added inline comments.


================
Comment at: lib/Transforms/IPO/FunctionImport.cpp:454
       for (auto Ref : Summary->refs()) {
         if (LiveSymbols.insert(Ref).second) {
           DEBUG(dbgs() << "Marking live (ref): " << Ref.getGUID() << "\n");
----------------
Do we need the `LiveSymbols` set any more? Can we use `isLive` and `setLive` to mark summaries as live as we go?


Repository:
  rL LLVM

https://reviews.llvm.org/D33615