[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