[PATCH] D154314: [LV] Remove the reminder loop if we know the mask is always true

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 10 01:49:24 PDT 2023


dmgreen added a comment.

There are already function attributes for specifying the minimum and maximum vscale vector lengths through vscale_range. I think it should be expanded to include whether the vscale is know to be a power-of-two. That would allow Instcombine and value tracking to properly reason about it in other situations too, as well as removing the dead tails in cases like this.

That doesn't mean this patch is not a good idea too, as we already know the tail is unnecessary in the vectorizer and removing it earlier will have benefits.


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

https://reviews.llvm.org/D154314



More information about the llvm-commits mailing list