[PATCH] D19481: [ThinLTO] Use valueid instead of bitcode offsets in combined index file

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 13:34:20 PDT 2016


tejohnson added a comment.

In http://reviews.llvm.org/D19481#412694, @joker.eph wrote:

> A lot more lines removed than added... these are the best patches :)
>
> It seems quite mechanical, I skimmed through it and nothing struck me, so LGTM.


Thanks. Just note that I will wait until http://reviews.llvm.org/D18298 goes in and then update to use your new helper method findSummaryInModule from that patch.


http://reviews.llvm.org/D19481





More information about the llvm-commits mailing list