[PATCH] D74867: [MLIR] add out of tree version of mlir-opt

Stephen Neuendorffer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 26 09:12:28 PDT 2020


stephenneuendorffer added a comment.

In D74867#1943837 <https://reviews.llvm.org/D74867#1943837>, @marbre wrote:

> Out of curiosity, no need to call define and `registerTestPasses()`? Further, calling `registerAllDialects()` and `registerAllPasses()` can be skipped? Is that due linking against `libMLIR.so`?


No, this is just a little old.  Currently, calling registerAllDialects() and registerAllPasses() is required if you want any of the standard stuff. The .cpp code was just a copy of what's in tools/mlir-opt.  The critical part of this is the CMakeLists.txt, which was intended as a test for linking against libMLIR.so.  If you actually want to build an external project see: https://reviews.llvm.org/D76047


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74867





More information about the llvm-commits mailing list