llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-driver Author: Matt Arsenault (arsenm) <details> <summary>Changes</summary> Also remove redundant SPIRV check. --- Full diff: https://github.com/llvm/llvm-project/pull/189261.diff 1 Files Affected: - (modified) clang/lib/Driver/Driver.cpp (+1-1) ``````````diff diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp index 3e489f0b1f4cf..f67dfcf1d3609 100644 --- a/clang/lib/Driver/Driver.cpp +++ b/clang/lib/Driver/Driver.cpp @@ -4467,7 +4467,7 @@ shouldBundleHIPAsmWithNewDriver(const Compilation &C, if (!TC) continue; const llvm::Triple &Tr = TC->getTriple(); - if (Tr.isSPIRV() || Tr.getArch() != llvm::Triple::amdgcn) + if (!Tr.isAMDGPU()) return false; HasAMDGCNHIPDevice = true; } `````````` </details> https://github.com/llvm/llvm-project/pull/189261 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
