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

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 27 00:16:39 PDT 2017


eugenis planned changes to this revision.
eugenis added a comment.

Ouch. Thank you for bringing this to my attention.
Looking at the current cfi-icall patch, this combined summary / serialization code is dead indeed.
The patch somehow evolved to make the isLive() check in ThinLTO backend redundant without me noticing.
I'll revert the summary parts of this change, probably next week.


Repository:
  rL LLVM

https://reviews.llvm.org/D33615





More information about the llvm-commits mailing list