[PATCH] D109368: [LV] Vectorize cases with larger number of RT checks, execute only if profitable.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 3 03:06:28 PDT 2022
fhahn marked 2 inline comments as done.
fhahn added a comment.
In D109368#3545453 <https://reviews.llvm.org/D109368#3545453>, @nikic wrote:
> The patch title and description could use an update. It currently sounds like this limits the amount of vectorization, while in practice this makes vectorization much more aggressive (right?)
Thanks, I just updated the title & description!
> Just to double check, now that an additional limit has been introduced, did the large code size regressions go away?
Mostly yes, I updated the description with size numbers for AArch64 with -O3 (should be the same configuration used for the earlier numbers). There are still a few increases, but much less severe ones than originally. Also that's out of 237 binaries. (SPEC2006, SPEC2017rate and MultiSource)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109368/new/
https://reviews.llvm.org/D109368
More information about the llvm-commits
mailing list