[flang-commits] [clang] [flang] [lld] [lldb] [llvm] AMDGPU: Introduce amdgpu triple arch (PR #206480)

[Scott Linder via flang-commits]

================
@@ -874,6 +880,24 @@ AMDGPUTargetMachine::AMDGPUTargetMachine(const Target &T, const Triple &TT,
       TLOF(createTLOF(getTargetTriple())) {
   initAsmInfo();
   if (TT.isAMDGCN()) {
+    // Triple is missing a representation for non-empty, but unrecognized
----------------
slinder1 wrote:

Is it too much churn to expand the representation in Triple itself?

It seems like comparisons against `NoSubArch` for everyone else would just need to be for `NoSubArch || UnknownSubArch`?

Doesn't have to be done here, but it is a bit unfortunate to have to hack around it in this way

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