[PATCH] D129625: [NFC][SVE] Add tests for zext(cmpeq(x, splat(0)))

Peter Waller via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 07:57:24 PDT 2022


peterwaller-arm added a comment.

In D129625#3648235 <https://reviews.llvm.org/D129625#3648235>, @c-rhodes wrote:

> Remove FP tests. I noticed there was no ucvtf after the mov in some tests, I mis-generated them somehow, but I'm not entirely sure the fold to CNOT is correct for FP types anyway, so I've removed them.

It seems sensible to retain the tests with the correct check lines in any case. It may be possible to use a cnot if no-signed-zeros is in effect, but that's a patch for another day.


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

https://reviews.llvm.org/D129625



More information about the llvm-commits mailing list