[llvm] [DTLTO][LLVM] Integrated Distributed ThinLTO (DTLTO) (PR #127749)

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Fri May 16 09:54:03 PDT 2025


teresajohnson wrote:

> I would like to summarize the state of this PR as I understand it. Firstly, some reviewers have indicated that this PR is acceptable in its current state. However, there is an alternative proposal: https://github.com/romanova-ekaterina/llvm-project/pull/new/kromanova/main/integrated-DTLTO-no-backend. We somehow need to come to a conclusion, and I think an appeal to authority (fallacy or not) is the way :)
> 
> @MaskRay – can I please ask you to reply to @teresajohnson's comment: [#127749 (comment)](https://github.com/llvm/llvm-project/pull/127749#issuecomment-2789902438). @tru has created a distributor that works with his organization's fork of FASTBuild, using the JSON interface. If you prefer the no-backend approach, then the JSON interface from this PR could be merged into that approach.
> 
> Thanks.

ping @MaskRay. But overall, this is not a lot of code in LTO, and so like @cachemeifyoucan I have a slight preference for this one. It is good to hear that it has already been adopted successfully by another user.

I just went through again and have a few comments, but overall it is in good shape.

https://github.com/llvm/llvm-project/pull/127749


More information about the llvm-commits mailing list