[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