[PATCH] D106164: [LV] Don't assume isScalarAfterVectorization if one of the uses needs widening.
David Sherwood via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 23 05:55:01 PDT 2021
david-arm accepted this revision.
david-arm added a comment.
This revision is now accepted and ready to land.
LGTM! Could you address the nit before merging? Thanks. :)
================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/sve-widen-gep.ll:78
+!3 = !{!"llvm.loop.vectorize.scalable.enable", i1 true}
+!4 = !{ !5 }
+!5 = distinct !{ !5, !6 }
----------------
nit: I think you can delete !4 -> !7 now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106164/new/
https://reviews.llvm.org/D106164
More information about the llvm-commits
mailing list