[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