[Mlir-commits] [mlir] [mlir][cmake] Fix mlir target export (PR #153341)

[llvmlistbot at llvm.org]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: Boyana Norris (brnorris03)

<details>
<summary>Changes</summary>

In https://github.com/llvm/llvm-project/pull/152195, target export was accidentally moved inside a conditional, but it should have been left outside. This patch undoes that change.

---
Full diff: https://github.com/llvm/llvm-project/pull/153341.diff


1 Files Affected:

- (modified) mlir/cmake/modules/AddMLIR.cmake (+1-1) 


``````````diff
diff --git a/mlir/cmake/modules/AddMLIR.cmake b/mlir/cmake/modules/AddMLIR.cmake
index 14eefb50ca714..38839679ef8b1 100644
--- a/mlir/cmake/modules/AddMLIR.cmake
+++ b/mlir/cmake/modules/AddMLIR.cmake
@@ -641,8 +641,8 @@ function(add_mlir_library_install name)
                               COMPONENT ${name})
     endif()
     set_property(GLOBAL APPEND PROPERTY MLIR_ALL_LIBS ${name})
-    set_property(GLOBAL APPEND PROPERTY MLIR_EXPORTS ${name})
   endif()
+  set_property(GLOBAL APPEND PROPERTY MLIR_EXPORTS ${name})
 endfunction()
 
 # Declare an mlir library which is part of the public C-API.

``````````

</details>


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