[clang] [llvm] [clang][Driver] Add HIPAMD Driver support for AMDGCN flavoured SPIR-V (PR #95061)

Alex Voicu via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 10 17:13:40 PDT 2024


================
@@ -128,12 +128,13 @@ enum class CudaArch {
   GFX12_GENERIC,
   GFX1200,
   GFX1201,
+  AMDGCNSPIRV,
   Generic, // A processor model named 'generic' if the target backend defines a
            // public one.
   LAST,
 
   CudaDefault = CudaArch::SM_52,
-  HIPDefault = CudaArch::GFX906,
+  HIPDefault = CudaArch::AMDGCNSPIRV,
----------------
AlexVlx wrote:

In the long run it should not (the BE is almost baked, for example); in the interim, it will need the translator, and I (think I) do see what you are saying, that it might be onerous to have HIPAMD depend on that - I can unflip the default pending the BE coming along. The only cost would be slightly more dubious ergonomics, but those are pre-existing.

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


More information about the cfe-commits mailing list