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

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 7 17:08:08 PDT 2016


mehdi_amini added a comment.

In https://reviews.llvm.org/D25345#565040, @tejohnson wrote:

> But this change is needed for now to keep the current model working


"Keep the current model working" isn't exact: I believe it is broken since the recent addition of "false empty summaries" for module.
So have the current model working again now, you would need to revert this.


https://reviews.llvm.org/D25345





More information about the llvm-commits mailing list