[MLIR] Fix indexing maps in isColumnMajorMatmul

[GMNGeoffrey]

Summary

This patch updates the indexing maps used for checking against a
column-major matmul operation. The maps as currently written do not
describe a column-major matmul. The linalg named op column_major_matmul
has the correct maps (and notably fails this test).

If C = matmul(A, B) we want an operation that given A in column major
format and B in column major format produces C in column major format.
Given that for a matrix, faux column major is just transpose.
column_major_matmul(transpose(A), transpose(B)) = transpose(C). If A
is NxK and B is KxM, then C is NxM, so transpose(A) is
KxN, transpose(B) is MxK and transpose(C) is MxN, not NxM as
these maps currently have.

Longer-term, this check should be permutation-invariant to be more
general, but for now let's have it at least use a correct map to avoid
further confusion.

There are no longer any uses of this in the repo, but I expect to add further
uses as part of expanding linalg interfaces.

[GMNGeoffrey]

This has been subsumed by https://reviews.llvm.org/D96984

mlir/include/mlir/Dialect/Utils/StructuredOpsUtils.h

 }
 
 inline bool isColumnMajorMatmul(ArrayAttr indexingMaps) {
   auto context = indexingMaps.getContext();
   AffineExpr m, n, k;
   bindDims(context, m, n, k);
   auto mapA = AffineMapAttr::get(AffineMap::get(3, 0, {k, n}, context));
   auto mapB = AffineMapAttr::get(AffineMap::get(3, 0, {m, k}, context));
-  auto mapC = AffineMapAttr::get(AffineMap::get(3, 0, {n, m}, context));
+  auto mapC = AffineMapAttr::get(AffineMap::get(3, 0, {m, n}, context));
   auto maps = ArrayAttr::get(context, {mapA, mapB, mapC});
   return indexingMaps == maps;
 }
 
 /// Attribute name for the AffineArrayAttr which encodes the relationship
 /// between a structured op iterators' and its operands.
 constexpr StringRef getIndexingMapsAttrName() { return "indexing_maps"; }