[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