[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