================
@@ -9374,8 +9374,11 @@ void LinkerWrapper::ConstructJob(Compilation &C, const 
JobAction &JA,
       // For SPIR-V, pass some extra flags to `spirv-link`, the out-of-tree
       // SPIR-V linker. `spirv-link` isn't called in LTO mode so restrict these
       // flags to normal compilation.
-      if (TC->getTriple().isSPIRV() && !C.getDriver().isUsingLTO() &&
-          !C.getDriver().isUsingOffloadLTO()) {
+      // SPIR-V for AMD doesn't use spirv-link and therefore doesn't need these
----------------
sarnex wrote:

Thanks for the fix! It would be great if we could add a test. Even if the only 
symptom is an assertion, at least it would be caught in builds with assertions 
enabled

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

Reply via email to