================ @@ -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