[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