[PATCH] D105632: [LV] Use lookThroughAnd with logical reductions

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 21 08:59:37 PDT 2021


spatel added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/trunc-reductions.ll:237
+  %sum.02p = phi i32 [ %min, %for.body ], [ 0, %entry ]
+  %sum.02 = and i32 %sum.02p, 65535
+  %gep = getelementptr inbounds i16, i16* %ptr, i32 %iv
----------------
It's independent of this patch, but I noticed that all of the affected tests have this kind of mask op on the phi. 
I think -instcombine will eliminate that op, so -loop-vectorize will not typically see this pattern, and we still get the wide reduction op. Is there a plan to account for that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105632



More information about the llvm-commits mailing list