[flang-commits] [clang] [flang] [llvm] clang: Start using new amdgpu subarch triples (PR #206482)

[Joseph Huber via flang-commits]

================
@@ -1124,11 +1125,17 @@ ToolChain::getTargetSubDirPath(StringRef BaseDir) const {
   if (auto Path = getPathForTriple(T))
     return *Path;
 
-  // Handle the legacy AMDGPU triple case as well.
-  if (T.getArchName() == "amdgcn") {
-    llvm::Triple Canon(T);
-    Canon.setArchName("amdgpu");
-    if (auto Path = getPathForTriple(Canon))
+  if (T.isAMDGCN()) {
+    // Clear the subarch as a fallback.
+    // TODO: Remove this when libc and compiler-rt builds are migrated.
----------------
jhuber6 wrote:

Migrated how? I think the cache files already suggest the new triples, but it's mostly up to the user and they're ideally backward compatible from the aliases I put in the library search logic.

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