[PATCH] D25345: [ThinLTO] Handle empty summaries during internalization

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 11 14:57:56 PDT 2016


mehdi_amini added a comment.

But my issue is not what `moduleCanBeRenamedForThinLTO()` does right now, but rather what do we do when it returns "false". These modules previously didn't go through the ThinLTO pipeline, so it was fine. Now they have an empty summary and they go to the thin-link. This seems an issue to me, and I rather have a proper answer design-wise rather than patching over-and-over as bugs are reported.


https://reviews.llvm.org/D25345





More information about the llvm-commits mailing list