Henry =?utf-8?q?Linjamäki?= <[email protected]>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>


================
@@ -6990,6 +6990,12 @@ const ToolChain &Driver::getToolChain(const ArgList 
&Args,
         break;
       case llvm::Triple::spirv32:
       case llvm::Triple::spirv64:
+        if (Target.getOSName() == "hipspv" ||
----------------
jhuber6 wrote:

Why do we need this here? The linker wrapper passes 
`--target=spirv64-amd-amdhsa` which keys off of the HSA OS above. Do we need 
separate handling?

https://github.com/llvm/llvm-project/pull/168043
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to