banach-space wrote: Thanks for taking a look! > The getNumDynamicDims in SparseTensorType should also be updated for consistency. > And what about getNumScalableDims in vector type? Sending update shortly 👍🏻 https://github.com/llvm/llvm-project/pull/110472