[PATCH] D20268: [wip] Resolution-based LTO API.

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Thu May 19 19:15:14 PDT 2016


pcc added a comment.

> That was my recommendation in the reviews on the earlier version of this patch, but a comment on the LTOBackend class made me think pcc planned to use this for the distributed backends themselves too (see the comment I noted below).


Maybe I misunderstood you then. My understanding from this:

> > Now that I think about it, there's probably no need to use the LTO class itself in the distributed backends.

> 

> 

> Yes and I think it shouldn't be in the same library, but rather in Transforms/IPO.


was that we should move the code that implements the distributed backend to IPO. That's what LTOBackend is.


http://reviews.llvm.org/D20268





More information about the llvm-commits mailing list