[PATCH] D129221: [RISCV] Switch to using get.active.lane.mask when tail folding

Thu Jul 7 13:51:59 PDT 2022

craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

In D129221#3636023 <https://reviews.llvm.org/D129221#3636023>, @reames wrote:

> In D129221#3633717 <https://reviews.llvm.org/D129221#3633717>, @craig.topper wrote:
>
>> I don't see a vector IV being killed off in any of the changed tests. Am I missing something?
>
> No, you're completely right, I got myself confused.  The vectorizer internally represents this with a test on a newly introduced vector IV, but then later generation handling (still in the vectorizer) converts that back into a use of the scalar one + a broadcast.  So the actual code generated doesn't involve the vector IV at all.

Thanks for clarifying. LGTM

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129221