[PATCH] D72586: [MLIR] Move from add_llvm_library to add_llvm_component_library

[Valentin Churavy via Phabricator via llvm-commits]

vchuravy added a comment.

In D72586#1825783 <https://reviews.llvm.org/D72586#1825783>, @mehdi_amini wrote:

> Are there other LLVM subproject who inject themselves into the global llvm shared library? Should we have a libMLIR.so instead?


Yes I think a libMLIR.so is the right choice, the only reason to go the other way would be if LLVM starts depending on MLIR. A libMLIR.so is a clearer separation and even LTO is split out into its own library.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72586