[PATCH] D101260: [LoopVectorize][SVE] Remove assert for scalable vector in InnerLoopVectorizer::fixReduction
David Sherwood via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 26 01:32:13 PDT 2021
david-arm added a comment.
The fix looks good thanks! Just a minor comment about the test ...
================
Comment at: llvm/test/Transforms/LoopVectorize/sve-reduction-inloop.ll:12
+; CHECK-NEXT: %[[ICMP1:.*]] = icmp eq i32 %[[INDEXNEXT]], %{{.*}}
+; CHECK-NEXT: %[[TRUNC1:.*]] = trunc <vscale x 8 x i32> %{{.*}} to <vscale x 8 x i8>
+; CHECK-NEXT: %[[ZEXT1:.*]] = zext <vscale x 8 x i8> %[[TRUNC1]] to <vscale x 8 x i32>
----------------
Hi @CarolineConcatto, I presume this is a truncation of the PHI value? If possible I think it might be nice to have CHECK lines for the PHI values to see them being truncated. I think that ZEXT1 and ZEXT2 will also be the incoming values for the PHI node too.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101260/new/
https://reviews.llvm.org/D101260
More information about the llvm-commits
mailing list