[PATCH] D88962: [SVE] Add support for scalable vectors in vectorize_width loop attribute
David Sherwood via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 8 00:52:08 PDT 2020
david-arm added inline comments.
================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp:67
+ return false;
+ const ConstantInt *C0 =
+ mdconst::dyn_extract<ConstantInt>(MD->getOperand(0));
----------------
fhahn wrote:
> are you anticipating additional hints with ElementCount metadata? If not, it might be simpler to just deal with HK_WIDTH up front and validate Val[0] and Val[1] here and leave the code handling the other cases mostly unchanged? Or maybe use named variables instead of an array, to make things a bit clearer?
So the reason I structured it like this is because we are still maintaining backwards compatibility with the old style and allowing single integer constants instead of the node. I could deal with the HK_WIDTH up front, but I'd still need all this code. Would you be ok with using named variables instead?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88962/new/
https://reviews.llvm.org/D88962
More information about the llvm-commits
mailing list