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

Steven Wu via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 10 10:27:43 PDT 2025


cachemeifyoucan wrote:

I don't have a strong opinion for either implementation. If I have to pick one, I like this PR better. I feel like the alternative will require keeping more implementation details of the build system in tree of LLVM. That is not a problem for itself but I don't know how stable those APIs and implementation details are, so it might be a problem for integration.

On the other side, I see this JSON interface from this PR is very simple and stable (maybe we should add a version in there just to be safe). I don't think the LTO interface is bloated after the change and it is not stable anyway. Thus slightly prefer this patch. 

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


More information about the llvm-commits mailing list