afonso360 opened PR #4331 from aarch64-breduce-ireduce-isle
to main
:
Hey, This moves
ireduce
andbreduce
over to ISLE. Its pretty much a copy of the x64 backend since both implementations are equal.
akirilov-arm submitted PR review.
akirilov-arm created PR review comment:
ty_int_bool_ref_scalar_64
is a more accurate choice thanfits_in_64
.
akirilov-arm submitted PR review.
akirilov-arm created PR review comment:
Ditto.
afonso360 updated PR #4331 from aarch64-breduce-ireduce-isle
to main
.
afonso360 closed without merge PR #4331.
afonso360 reopened PR #4331 from aarch64-breduce-ireduce-isle
to main
.
akirilov-arm created PR review comment:
Were these changes prompted by CI issues?
akirilov-arm submitted PR review.
afonso360 submitted PR review.
afonso360 created PR review comment:
Oops, left those in by accident, sorry.
afonso360 updated PR #4331 from aarch64-breduce-ireduce-isle
to main
.
akirilov-arm created PR review comment:
I don't think that
in the platform-defined way
is a useful specification, but the operation is equivalent to the integer one, so you can just repeat the same statement here, i.e.by discarding the most significant bits
.
akirilov-arm submitted PR review.
akirilov-arm submitted PR review.
afonso360 updated PR #4331 from aarch64-breduce-ireduce-isle
to main
.
akirilov-arm submitted PR review.
cfallin submitted PR review.
cfallin merged PR #4331.
Last updated: Dec 23 2024 at 12:05 UTC