[PATCH] D67223: [LLVM][Alignment] Convert isLegalNTStore/isLegalNTLoad to llvm::Align

[Clement Courbet via Phabricator via llvm-commits]

courbet added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp:741
           assert(VecTy && "did not find vectorized version of stored type");
           unsigned Alignment = getLoadStoreAlignment(ST);
+          if (!TTI->isLegalNTStore(VecTy, llvm::Align(Alignment))) {
----------------
Given that `getLoadStoreAlignment` is eventually going to return a MaybeAlign, I would assert here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67223/new/

https://reviews.llvm.org/D67223