[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