[PATCH] D88962: [SVE] Add support for scalable vectors in vectorize_width loop attribute

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 27 07:44:54 PDT 2020


david-arm added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/metadata-width.ll:55
+!3 = !{!"llvm.loop.vectorize.width", !4}
+!4 = !{i32 8, i32 0}
----------------
david-arm wrote:
> sdesmalen wrote:
> > nit: `s/i32 0/i1 false/`
> If I make this change the test fails. We will still always print out "i32 0". Is there a way to control the format generated?
Sorry I just realised this is an input to the test, rather than testing the output! Please ignore my comment.


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

https://reviews.llvm.org/D88962



More information about the llvm-commits mailing list