[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