akirilov-arm opened PR #3383 from vany_true
to main
:
Fixes #3305.
akirilov-arm updated PR #3383 from vany_true
to main
.
cfallin submitted PR review.
cfallin submitted PR review.
cfallin created PR review comment:
I'm possibly misunderstanding, but doesn't this only work for the "any" case, not the "all" case, since in the latter case half the lanes will be new artificial zeroes (possibly, if the upper bits of the i64 lanes are zeroes)?
akirilov-arm submitted PR review.
akirilov-arm created PR review comment:
Yes, you are absolutely right, but there is a separate arm for
VallTrue
with 64-bit elements on line 2343.
cfallin submitted PR review.
cfallin submitted PR review.
cfallin created PR review comment:
Ah, I missed that, thank you! Could you add a comment referring to the case above and noting that
VallTrue
with 64x2 doesn't land here?
akirilov-arm updated PR #3383 from vany_true
to main
.
cfallin submitted PR review.
cfallin merged PR #3383.
Last updated: Dec 23 2024 at 12:05 UTC