[clang] [llvm] [clang][driver][darwin] Base the system prefix on the SDK, not the target (PR #171970)

Ian Anderson via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 11 22:58:57 PST 2025


================
@@ -181,6 +192,7 @@ class DarwinSDKInfo {
   VersionTuple Version;
   VersionTuple MaximumDeploymentTarget;
   llvm::Triple::OSType OS;
+  std::unordered_map<llvm::Triple, std::string> SystemPrefixes;
----------------
ian-twilightcoder wrote:

I'm not really sure if this makes better sense as a `SmallDenseMap` or an `unordered_map`. I tried both (the first commit uses `SmallDenseMap` and the second replaces it with `unordered_map`) and they both work. `unordered_map` is a little less code in `Triple` to make it work, and @felipepiovezan suggested that was better because `Triple` might be a bit large to efficiently use as a `SmallDenseMap` key.

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


More information about the llvm-commits mailing list