================
@@ -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,
----------------
yxsamliu wrote:

Did this patch pass our internal PSDB?

I would recommend deferring changing the default offload arch to future while 
checkin the other changes first.

I doubt the downstream build is ready to have llvm-link and llvm-spirv. Some 
HIP apps do a test compilation of HIP program with default offload arch and 
this will break them.

https://github.com/llvm/llvm-project/pull/95061
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to