[clang] [HIP][SPIRV] Enable the SPIRV backend instead of the translator through an experimental flag. (PR #162282)
Manuel Carrasco via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 8 03:35:33 PDT 2025
Juan Manuel Martinez =?utf-8?q?CaamaƱo?= <juamarti at amd.com>,Manuel
Carrasco <Manuel.Carrasco at amd.com>,Manuel Carrasco <Manuel.Carrasco at amd.com>,Manuel
Carrasco <Manuel.Carrasco at amd.com>,Manuel Carrasco <Manuel.Carrasco at amd.com>,Manuel
Carrasco <Manuel.Carrasco at amd.com>,Manuel Carrasco <Manuel.Carrasco at amd.com>,Manuel
Carrasco <Manuel.Carrasco at amd.com>,Manuel Carrasco <Manuel.Carrasco at amd.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/162282 at github.com>
================
@@ -995,7 +995,7 @@ inferOffloadToolchains(Compilation &C, Action::OffloadKind Kind) {
StringRef Triple;
if (ID == OffloadArch::AMDGCNSPIRV)
- Triple = "spirv64-amd-amdhsa";
+ Triple = "spirv64v1.6-amd-amdhsa";
----------------
mgcarrasco wrote:
@shiltian thanks for all the feedback. I refactored the code so we have `getSpirvBackendTriple()` which returns the hardcoded triple for the two uses we have (here and in the [Linker](https://github.com/llvm/llvm-project/pull/162282/files#diff-242dcb66348f860d8610bab6cf633f3bc158620853e0987ec2f1b47ee982719cR189)). We could also write a function that returns the harcoded version, similar to `getSpirvBackendTriple()`. Let me know what you think.
https://github.com/llvm/llvm-project/pull/162282
More information about the cfe-commits
mailing list