[Mlir-commits] [mlir] [mlir][test] Make SME e2e tests require an emulator (PR #86489)

Mehdi Amini llvmlistbot at llvm.org
Tue Mar 26 14:23:33 PDT 2024


joker-eph wrote:

> No lines is better than 3

This does not seem like a useful comment IMO.

A nice error message at configuration time is much better than a compile time failure, which is itself much better than a link-time failure, which itself is much better than a runtime test failure!

There is much more than "3 lines" in terms of complexity that exists in MLIR as safeguard against user mis-configuration.
For example detecting creation of operation or attribute when the dialect isn't loaded, or more recently the whole "promised" interfaces: 
https://github.com/llvm/llvm-project/blob/80487e1c622d51f4c0df12b64cca8a0b346e9b85/mlir/include/mlir/IR/OpDefinition.h#L2062-L2068
(same code exists in other places)
Leading to: https://github.com/llvm/llvm-project/blob/80487e1c622d51f4c0df12b64cca8a0b346e9b85/mlir/include/mlir/IR/Dialect.h#L231-L243

> I've seen setups in the past for example where emulation is enabled opaquely via binfmt_misc on Linux.

That seems like a valid use-case, @banach-space how do you see this supported?
Also: what about someone who has the actual HW in the future?


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


More information about the Mlir-commits mailing list