[mlir] Fix sporadic build failures due to missing dependency
authorStella Stamenova <stilis@microsoft.com>
Tue, 13 Oct 2020 16:53:07 +0000 (09:53 -0700)
committerStella Stamenova <stilis@microsoft.com>
Tue, 13 Oct 2020 16:53:07 +0000 (09:53 -0700)
The build of MLIR occasionally fails (especially on Windows) because there is missing dependency between MLIRLLVMIR and MLIROpenMPOpsIncGen.

1) LLVMDialect.cpp includes LLVMDialect.h
2) LLVMDialect.h includes OpenMPDialect.h
3) OpenMPDialect.h includes OpenMPOpsDialect.h.inc, OpenMPOpsEnums.h.inc and OpenMPOps.h.inc

The OpenMP .inc files are generated by MLIROpenMPOpsIncGen, so MLIRLLVMIR which builds LLVMDialect.cpp should depend on MLIROpenMPOpsIncGen

Reviewed By: mehdi_amini

Differential Revision: https://reviews.llvm.org/D89275

mlir/lib/Dialect/LLVMIR/CMakeLists.txt

index 9827b86..db1a5c4 100644 (file)
@@ -11,6 +11,7 @@ add_mlir_dialect_library(MLIRLLVMIR
   DEPENDS
   MLIRLLVMOpsIncGen
   MLIRLLVMConversionsIncGen
+  MLIROpenMPOpsIncGen
   intrinsics_gen
 
   LINK_COMPONENTS