[Mlir-commits] [mlir] [MLIR][AArch64] Lower vector.contract to Neon FEAT_BF16 operations (PR #148198)

[Andrzej Warzyński]

https://github.com/banach-space commented:

Thanks for working on this, it's exciting that we will finally have this implemented upstream 🙏🏻 

I've skimmed through and it mostly makes sense - this is very similar to what you did for SVE patterns and feels very familiar, nice!

I've left one inline comment, but also suggest splitting this into two PRs - each change in this PR deserves a dedicated commit in MLIR (as in, both changes are large and self-contained and to me it makes more sense to merge them separately).

Could you split this PR? I am unlikely to have other comments beyond this.

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