[Mlir-commits] [mlir] [mlir][tosa] Convert group tosa::Conv2DOp to linalg conv (PR #108192)

[Georgios Pinitas]

GeorgeARM wrote:

>From what I gather so far the TOSA specification does not define anything around grouped convolutions in the Conv2d operator itself. Not quite sure if there are plans to be added or not @eric-k256 and @sjarus will be able to provide more information. If we are happy to deviate from the spec with an optional attribute I can proceed with reviewing this patch.

In the meantime may I suggest to split the linalg grouped conv structured operations to a separate patch and keep the TOSA dialect extension and legalisation in this one. This way we can merge the linalg part faster?

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