[Mlir-commits] [mlir] [mlir][CAPI][python] expose the python bindings for linalg::isaConvolutionOpInterface and linalg::inferConvolutionDims (PR #135253)

[Bangtian Liu]

bangtianliu wrote:

> this PR seems to cause breakage in downstream CAPI users
> 
> ```c++
> /opt/hostedtoolcache/Python/3.11.11/x64/lib/python3.11/site-packages/mlir/lib/../include/mlir-c/Dialect/Linalg.h:34:20: error: must use 'struct' tag to refer to type 'MlirLinalgContractionDimensions'
> MLIR_CAPI_EXPORTED MlirLinalgContractionDimensions
>                    ^
>                    struct 
> /opt/hostedtoolcache/Python/3.11.11/x64/lib/python3.11/site-packages/mlir/lib/../include/mlir-c/Dialect/Linalg.h:50:20: error: must use 'struct' tag to refer to type 'MlirLinalgConvolutionDimensions'
> MLIR_CAPI_EXPORTED MlirLinalgConvolutionDimensions
>                    ^
>                    struct 
> ```

Thanks for pointing that out. I've just submitted a PR to fix it.

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