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

[Jakub Kuderski]

================
@@ -1729,18 +1729,18 @@ def Vector_LoadOp : Vector_Op<"load", [
                    "Value":$base,
                    "ValueRange":$indices,
                    CArg<"bool", "false">:$nontemporal,
-                   CArg<"uint64_t", "0">:$alignment), [{
+                   CArg<"llvm::Align", "llvm::Align()">:$alignment), [{
----------------
kuhar wrote:

Ah right, I didn't realize that `llvm::Align` defaults to 0 but for the shift amount, not the alignment value. `llvm::MaybeAlign` sounds like a better type then.

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