[llvm] [mlir] [mlir][spirv] Fix int type declaration duplication when serializing (PR #145687)

Davide Grohmann via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 1 13:11:44 PDT 2025


================
@@ -5,8 +5,8 @@ if (NOT LLVM_INCLUDE_SPIRV_TOOLS_TESTS)
   return()
 endif ()
 
-if (NOT "SPIRV" IN_LIST LLVM_TARGETS_TO_BUILD)
-  message(FATAL_ERROR "Building SPIRV-Tools tests is unsupported without the SPIR-V target")
+if (NOT "SPIRV" IN_LIST LLVM_TARGETS_TO_BUILD AND NOT "mlir" IN_LIST LLVM_ENABLE_PROJECTS)
----------------
davidegrohmann wrote:

Ok, if I understand you correctly I think this `if` can be dropped completely. The CMake flag to enable spirv-tools is `LLVM_INCLUDE_SPIRV_TOOLS_TESTS` and it is tested at the beginning of this file.

https://github.com/llvm/llvm-project/pull/145687


More information about the llvm-commits mailing list