[PATCH] D30247: Epilog loop vectorization
Ashutosh Nema via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 7 02:38:55 PDT 2017
ashutosh.nema added a comment.
This patch is little old and not updated with latest vectorization changes.
In the RFC there are few concerns raised around the new block layout:
a) If original loop alias check fails, then directly jump to scalar loop
b) On critical path there should not be extra checks, i.e. when minimum iteration check fails for the original vector loop then directly jump to scalar loop. In my opinion this is possible optimization opportunity loss, because when minimum iteration check for original loop fails, can still try executing epilog loop and that requires epilog loop’s minimum iteration check. We can keep his under option.
I guess both the cases can be handled, I’ll re initiate the discussion & update this patch soon.
Repository:
rL LLVM
https://reviews.llvm.org/D30247
More information about the llvm-commits
mailing list