[Mlir-commits] [mlir] [MLIR][CAPI][python] expose the python binding for linalgOp.getIndexingMaps (PR #136054)

Maksim Levental llvmlistbot at llvm.org
Thu Apr 17 08:38:04 PDT 2025


================
@@ -121,10 +121,15 @@ static void populateDialectLinalgSubmodule(nb::module_ m) {
   m.def("infer_convolution_dimensions", &InferConvolutionDimensions,
         "Infers convolution dimensions", nb::arg("op"));
 
-  m.def("get_indexing_maps_attr", &mlirLinalgGetIndexingMapsAttribute,
-        "Returns the indexing_maps or memoized_indexing_maps attribute for a "
-        "Linalg op.",
-        nb::arg("op"));
+  m.def(
+      "get_indexing_maps_attr",
----------------
makslevental wrote:

nit: i didn't notice this before but you don't need `_attr` here - in python i think usually you don't put types in the names of methods/functions unless you specifically want to distinguish between two functions that need to take (or return) different types

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


More information about the Mlir-commits mailing list