[PATCH] D98240: [VectorCombine] Simplify to scalar store if only one element updated

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 1 09:08:14 PDT 2021


fhahn added a comment.

In D98240#2723175 <https://reviews.llvm.org/D98240#2723175>, @qiucf wrote:

> Update tests and store size constraint. Thanks for explanation from @bjope and @fhahn

Thanks for the update! I think the only thing missing is a few tests for un-common vector types, like `<8 x i4>` or `<4 x i31>`. Probably worth to have at least 2 different tests with different types.


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

https://reviews.llvm.org/D98240



More information about the llvm-commits mailing list