[PATCH] D96522: [LV] Try larger VFs if VF is unprofitable for small types.
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 12 01:47:47 PST 2021
dmgreen added a comment.
OK, More about the loads being expensive than the reductions. If this is for 4 x i8 loads being expensive, should we just be overriding shouldMaximizeVectorBandwidth on AArch64? It could be based on SmallestType / WidestType if that makes it more precise, but I don't know that it needs to be.
https://godbolt.org/z/xG55W6
That would prevent us from putting this into the vectorizer costmodel directly, which may not be correct with the made up alignments/address space, and no indication of whether the loads are extended or not.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96522/new/
https://reviews.llvm.org/D96522
More information about the llvm-commits
mailing list