[PATCH] D129851: [AArch64][SVE] Consider more intrinsics in 'isZeroingInactiveLanes'.

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 15 08:56:31 PDT 2022


paulwalker-arm added a comment.

In D129851#3654860 <https://reviews.llvm.org/D129851#3654860>, @sdesmalen wrote:

> I didn't add new tests for the fcmp* cases, because there were no existing tests for these yet. The intrinsics seemed sensible to include in the switch statement. Let me know if you specifically want me to add new tests for these.

If there's functionality that we're relying on then I'd rather see a test to protect it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129851



More information about the llvm-commits mailing list