[PATCH] D73655: [MLIR] Break cyclic dependencies with MLIRAnalysis

[Stephen Neuendorffer via Phabricator via llvm-commits]

stephenneuendorffer retitled this revision from "[MLIR] Break cyclic dependency between MLIRPass and MLIRAnalysis" to "[MLIR] Break cyclic dependencies with MLIRAnalysis".
stephenneuendorffer edited the summary of this revision.
stephenneuendorffer updated this revision to Diff 242261.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73655/new/

https://reviews.llvm.org/D73655

Files:
  mlir/lib/Analysis/CMakeLists.txt
  mlir/lib/Dialect/VectorOps/CMakeLists.txt
  mlir/lib/Pass/CMakeLists.txt


Index: mlir/lib/Pass/CMakeLists.txt
===================================================================
--- mlir/lib/Pass/CMakeLists.txt
+++ mlir/lib/Pass/CMakeLists.txt
@@ -5,5 +5,5 @@
   ADDITIONAL_HEADER_DIRS
   ${MLIR_MAIN_INCLUDE_DIR}/mlir/Pass
   )
-add_dependencies(MLIRPass MLIRAnalysis MLIRIR LLVMSupport)
-target_link_libraries(MLIRPass MLIRAnalysis MLIRIR LLVMSupport)
+add_dependencies(MLIRPass MLIRVerifier MLIRIR LLVMSupport)
+target_link_libraries(MLIRPass MLIRVerifier MLIRIR LLVMSupport)
Index: mlir/lib/Dialect/VectorOps/CMakeLists.txt
===================================================================
--- mlir/lib/Dialect/VectorOps/CMakeLists.txt
+++ mlir/lib/Dialect/VectorOps/CMakeLists.txt
@@ -11,4 +11,4 @@
 add_dependencies(MLIRVectorOps MLIRVectorOpsIncGen)
 add_dependencies(MLIRVectorOps MLIRVectorTransformPatternsIncGen)
 
-target_link_libraries(MLIRVectorOps MLIRAnalysis MLIRIR MLIRStandardOps MLIRAffineOps MLIRLoopOps)
+target_link_libraries(MLIRVectorOps MLIRIR MLIRStandardOps MLIRAffineOps MLIRLoopOps MLIRLoopAnalysis)
Index: mlir/lib/Analysis/CMakeLists.txt
===================================================================
--- mlir/lib/Analysis/CMakeLists.txt
+++ mlir/lib/Analysis/CMakeLists.txt
@@ -1,4 +1,4 @@
-add_llvm_library(MLIRAnalysis STATIC
+set(LLVM_OPTIONAL_SOURCES
   AffineAnalysis.cpp
   AffineStructures.cpp
   CallGraph.cpp
@@ -14,6 +14,17 @@
   TestParallelismDetection.cpp
   Utils.cpp
   Verifier.cpp
+  )
+
+add_llvm_library(MLIRAnalysis
+  CallGraph.cpp
+  InferTypeOpInterface.cpp
+  Liveness.cpp
+  MemRefBoundCheck.cpp
+  OpStats.cpp
+  SliceAnalysis.cpp
+  TestMemRefDependenceCheck.cpp
+  TestParallelismDetection.cpp
 
   ADDITIONAL_HEADER_DIRS
   ${MLIR_MAIN_INCLUDE_DIR}/mlir/Analysis
@@ -24,4 +35,34 @@
   MLIRTypeInferOpInterfaceIncGen
   MLIRLoopOps
   )
-target_link_libraries(MLIRAnalysis MLIRAffineOps MLIRLoopOps)
+
+target_link_libraries(MLIRAnalysis MLIRPass MLIRAffineOps MLIRLoopOps)
+
+add_llvm_library(MLIRLoopAnalysis
+  AffineAnalysis.cpp
+  AffineStructures.cpp
+  LoopAnalysis.cpp
+  NestedMatcher.cpp
+  Utils.cpp
+
+  ADDITIONAL_HEADER_DIRS
+  ${MLIR_MAIN_INCLUDE_DIR}/mlir/Analysis
+  )
+add_dependencies(MLIRLoopAnalysis
+  MLIRAffineOps
+  MLIRCallOpInterfacesIncGen
+  MLIRTypeInferOpInterfaceIncGen
+  MLIRLoopOps
+  )
+
+target_link_libraries(MLIRLoopAnalysis MLIRAffineOps MLIRLoopOps)
+
+add_llvm_library(MLIRVerifier
+  Dominance.cpp
+  Verifier.cpp
+
+  ADDITIONAL_HEADER_DIRS
+  ${MLIR_MAIN_INCLUDE_DIR}/mlir/Analysis
+  )
+target_link_libraries(MLIRVerifier MLIRAffineOps MLIRLoopOps MLIRVectorOps)
+


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D73655.242261.patch
Type: text/x-patch
Size: 2605 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200204/a77ac80f/attachment.bin>