[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