[Mlir-commits] [llvm] [mlir] [CI] Add check-mlir-python to MLIR pre-merge checks (PR #72847)

Mehdi Amini llvmlistbot at llvm.org
Wed Nov 22 03:42:02 PST 2023


joker-eph wrote:

> MLIR could define its own jobs that run appropriate checks and use appropriate dependencies.

Are you suggesting that we should split the monolithic config? 
That seems like a pretty bad idea for the mono-repo in terms of efficiencies: LLVM is pretty "heavy" to build and if I change MLIR that means right now running both MLIR tests and Flang tests. If Flang and MLIR have disjoint testing that means that we would build LLVM+MLIR twice (once for a MLIR job and once for the Flang job).
The monolithic config is just nicely setup to build just the set of needed projects once, and run the necessary tests.

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


More information about the Mlir-commits mailing list