[PATCH] D67223: [LLVM][Alignment] Convert isLegalNTStore/isLegalNTLoad to llvm::Align
Clement Courbet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 5 05:43:33 PDT 2019
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
More information about the llvm-commits
mailing list