From 797adb027ee76d72a9240fdd830351994b0fd10b Mon Sep 17 00:00:00 2001
From: Ian Bearman <ianb@microsoft.com>
Date: Tue, 15 Feb 2022 10:02:37 -0800
Subject: [PATCH] This change adds a missing dependency (`MLIRIR`) to the
 onnx-mlir build in a few places. My team found this while investigating
 building llvm-project and onnx-mlir in parallel using a single cmake command.
 (#1178)

I've chosen to add the over-arching `MLIRIR` target rather than individual dependencies to avoid having to update these as interfaces are added/removed/moved within the MLIR IR. Below is the exact list of dependencies i found while researching this problem:

    MLIRCallInterfacesIncGen
    MLIRCastInterfacesIncGen
    MLIRDataLayoutInterfacesIncGen
    MLIROpAsmInterfaceIncGen
    MLIRRegionKindInterfaceIncGen
    MLIRSideEffectInterfacesIncGen
    MLIRSymbolInterfacesIncGen
    MLIRTensorEncodingIncGen
  ###ResultTypeInferenceOpInterface.cpp/HasOnnxSubgraphOpInterface.cpp/SpecializedKernelOpInterface.cpp
    MLIRBuiltinTypeInterfacesIncGen
    MLIRTensorEncodingIncGen

    MLIREmitCAttributesIncGen
    MLIRGPUOpsAttributesIncGen
    MLIRGPUOpsEnumsGen
    MLIROpenACCOpsIncGen
    MLIRSparseTensorAttrDefsIncGen
    MLIRTosaStructsIncGen

Signed-off-by: Ian Bearman <ianb@microsoft.com>

Co-authored-by: Stella Stamenova <stilis@microsoft.com>
---
 src/Compiler/CMakeLists.txt  | 1 +
 src/Interface/CMakeLists.txt | 4 ++++
 2 files changed, 5 insertions(+)

diff --git a/src/Compiler/CMakeLists.txt b/src/Compiler/CMakeLists.txt
index d70d782840a6..5fafbd4c0b95 100644
--- a/src/Compiler/CMakeLists.txt
+++ b/src/Compiler/CMakeLists.txt
@@ -31,6 +31,7 @@ add_onnx_mlir_library(CompilerUtils
 
   DEPENDS
   ExternalUtil
+  MLIRIR
 
   INCLUDE_DIRS PRIVATE
   ${FILE_GENERATE_DIR}
diff --git a/src/Interface/CMakeLists.txt b/src/Interface/CMakeLists.txt
index a8340d269fca..a30373b8561c 100644
--- a/src/Interface/CMakeLists.txt
+++ b/src/Interface/CMakeLists.txt
@@ -9,6 +9,7 @@ add_onnx_mlir_library(OMShapeInferenceOpInterface
   ShapeInferenceOpInterface.cpp
 
   DEPENDS
+  MLIRIR
   OMShapeInferenceOpInterfaceIncGen
 
   LINK_LIBS PUBLIC
@@ -19,6 +20,7 @@ add_onnx_mlir_library(OMResultTypeInferenceOpInterface
   ResultTypeInferenceOpInterface.cpp
 
   DEPENDS
+  MLIRIR
   OMResultTypeInferenceOpInterfaceIncGen
 
   LINK_LIBS PUBLIC
@@ -29,6 +31,7 @@ add_onnx_mlir_library(OMHasOnnxSubgraphOpInterface
   HasOnnxSubgraphOpInterface.cpp
 
   DEPENDS
+  MLIRIR
   OMHasOnnxSubgraphOpInterfaceIncGen
 
   LINK_LIBS PUBLIC
@@ -39,6 +42,7 @@ add_onnx_mlir_library(OMSpecializedKernelOpInterface
   SpecializedKernelOpInterface.cpp
 
   DEPENDS
+  MLIRIR
   OMSpecializedKernelOpInterfaceIncGen
 
   LINK_LIBS PUBLIC