[Mlir-commits] [mlir] [MLIR] Implement emulation of static indexing subbyte type vector stores (PR #115922)

Andrzej WarzyƄski llvmlistbot at llvm.org
Thu Jan 16 09:52:19 PST 2025


================
@@ -312,16 +420,17 @@ struct ConvertVectorStore final : OpConversionPattern<vector::StoreOp> {
 
     auto loc = op.getLoc();
     auto convertedType = cast<MemRefType>(adaptor.getBase().getType());
-    Type oldElementType = op.getValueToStore().getType().getElementType();
-    Type newElementType = convertedType.getElementType();
+    auto valueToStore = cast<VectorValue>(op.getValueToStore());
+    auto oldElementType = valueToStore.getType().getElementType();
+    auto newElementType = convertedType.getElementType();
----------------
banach-space wrote:

[nit] Looks like `convertedType` is not needed.
```suggestion
    auto newElementType = cast<MemRefType>(adaptor.getBase().getType()).getElementType();
```

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


More information about the Mlir-commits mailing list