[PATCH] D109432: [LoopVectorize] Permit fixed-width epilogue loops for scalable vector bodies

[Bardia Mahjour via Phabricator via llvm-commits]

bmahjour added a comment.

> However, you make a good point about the possibility of having different recipes for the same loop depending upon whether it's scalable or fixed-width.

Even for non-scalable targets, this patch may cause the epilogue plan to be different from that of the main loop. One could argue that it may be desirable to have different plans (recipes) for the two in some cases, but at least the selection criteria should not be arbitrary (as it would be if we just look for *a* plan that matches the VF).

I'm still wondering why we currently create two separate plans one for the scalable VFs and one for the fixed-width VFs, instead of one plan that includes a union of fixed-width VFs and scalable VFs? If we had such a single plan, then the workarounds in this patch would not be necessary.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109432/new/

https://reviews.llvm.org/D109432