[PATCH] D97549: [RISCV] Improve support of LoopVectorizer for RISC-V Vector

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

The changes LGTM, thanks for adding a test case.

However, I think the commit title/message could be expanded upon somewhat. For example, *how* does it improve LoopVectorizer support? Since we've got both fixed-length and scalable vectorization, I think it would be helpful to clarify that this relates to fixed-length vectorization. More specifically, doesn't this change *enable* fixed-length vectorization, not just improve upon it?


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

https://reviews.llvm.org/D97549