[Mlir-commits] [mlir] [NFC] Make AggregateOpInterface part of mlir:: instead of linalg:: (PR #70089)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Mon Oct 30 16:53:07 PDT 2023


MaheshRavishankar wrote:

While we resolve this,  @Abhishek-Varma instead of moving this to `mlir/Interfaces/...` Maybe just split this particular interface out of `LinalgInterface.td` and add the `AggregateOpInterface.td` to `mlir/Dialect/Linalg/IR/` itself. I think the original issue we were hitting was that we would have to pull in all of LinalgInterface to include this which causes issues. Having a separate `.td`/`.h` and `.cpp` in `mlir/Linalg/*` gets around that for now till we decide if and when to move this interface out of Linalg.

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


More information about the Mlir-commits mailing list