[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