[Mlir-commits] [mlir] [mlir][vector] Add alignment attribute to vector operations. (PR #152507)

[Erick Ochoa Lopez]

================
@@ -1919,7 +1922,6 @@ def Vector_MaskedLoadOp :
     load operation. It must be a positive power of 2. The operation must access
     memory at an address aligned to this boundary. Violations may lead to
     architecture-specific faults or performance penalties.
-    A value of 0 indicates no specific alignment requirement.
----------------
amd-eochoalo wrote:

I originally wanted thought about removing this line since I imagined that the constructors using the `llvm::Maybe` align will be preferred, but I now believe that adding this line back makes more sense since there are other constructors as well and the actual value stored is an integer attribute. Thanks for pointing it out! https://github.com/llvm/llvm-project/pull/152507/commits/47db5b173ffaddbc7a723d79881017a9ae2818a0

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