[Mlir-commits] [mlir] [mlir][vector] Add alignment attribute to `maskedload` and `maskedstore` (PR #151690)

Jakub Kuderski llvmlistbot at llvm.org
Thu Aug 7 11:47:24 PDT 2025


================
@@ -1729,18 +1729,18 @@ def Vector_LoadOp : Vector_Op<"load", [
                    "Value":$base,
                    "ValueRange":$indices,
                    CArg<"bool", "false">:$nontemporal,
-                   CArg<"llvm::Align", "llvm::Align()">:$alignment), [{
+                   CArg<"llvm::MaybeAlign", "llvm::MaybeAlign()">:$alignment), [{
       return build($_builder, $_state, resultType, base, indices, nontemporal,
-                   alignment != llvm::Align() ? $_builder.getI64IntegerAttr(alignment.value()) :
+                   alignment != llvm::MaybeAlign() ? $_builder.getI64IntegerAttr(alignment.valueOrOne().value()) :
----------------
kuhar wrote:

Since `MaybeAlign` inherits from `std::optional`, we can do
```suggestion
                   alignment.has_value() ? $_builder.getI64IntegerAttr(*alignment) :
```

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


More information about the Mlir-commits mailing list