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

via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 13 10:22:29 PDT 2025


================
@@ -426,6 +460,7 @@ class LTO {
     // The bitcode modules to compile, if specified by the LTO Config.
     std::optional<ModuleMapType> ModulesToCompile;
     DenseMap<GlobalValue::GUID, StringRef> PrevailingModuleForGUID;
+    DenseMap<StringRef, std::string> ModuleTriples;
----------------
bd1976bris wrote:

The backend now uses the first Triple in the link. I used the same Triple that regular LTO chooses. I have made this choice of triple available to the DTLTO backend in LLVM's `LTO.cpp`, so LLD does not need to explicitly choose a triple. I think this is a good choice because, due to the changes for multiple symbol tables for ARM64X for COFF LLD (https://github.com/llvm/llvm-project/commit/b068f2fd0fefca1ee357483333f034d18e6d8214), it's not quite as trivial for LLD to choose a Triple in some cases now. Note that it's easily doable - I just think it's simpler to have the code in LLVM's `LTO.cpp` on balance.

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


More information about the llvm-commits mailing list