[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