[PATCH] D84951: [LV] Try to sink users recursively for first-order recurrences.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 15 08:14:08 PDT 2020


fhahn added a comment.

In D84951#2219722 <https://reviews.llvm.org/D84951#2219722>, @xbolva00 wrote:

> https://godbolt.org/z/nXCxW8
>
> This should help to vectorize this code as well, or?

I think the unidentified PHI node in the example above is a pointer induction, not a first order recurrence? Looks like the actual problem is that LV only sees the inner loop fully unrolled and the SLP vectorizer does not vectorize the body of the unrolled loop. If built with `-fno-unroll-loops`, LV will vectorize the inner loop, but it would probably be better if the SLP vectorizer can just vectorize the unrolled body.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84951



More information about the llvm-commits mailing list