Skip to content

Revert shuffle-based Vector.Reverse implementation - #6

Merged
hez2010 merged 1 commit into
vector-lanesfrom
copilot/revert-commit-c6406092
May 14, 2026
Merged

Revert shuffle-based Vector.Reverse implementation#6
hez2010 merged 1 commit into
vector-lanesfrom
copilot/revert-commit-c6406092

Conversation

Copilot AI commented May 14, 2026

Copy link
Copy Markdown

This reverts c64060925b53a7ef24a796ed6e8c0e67ff95d49c, removing the shuffle-based Vector.Reverse<T> implementation introduced for the intrinsic vector types. The change restores the prior element-wise reversal behavior across the affected widths.

  • Scope

    • Reverted the Reverse<T> changes in:
      • Vector64
      • Vector128
      • Vector256
      • Vector512
  • Behavior

    • Restored the previous implementation strategy:
      • Vector64<T> uses the element-wise reverse loop
      • Wider vectors compose reversal from the next smaller vector width
  • Result

    • Removes the recently introduced shuffle-based path without changing unrelated intrinsic APIs or helpers.

This reverts commit c640609.

Co-authored-by: hez2010 <14960345+hez2010@users.noreply.github.com>
@hez2010
hez2010 marked this pull request as ready for review May 14, 2026 12:13
@hez2010
hez2010 merged commit 3f6a866 into vector-lanes May 14, 2026
1 check passed
@hez2010
hez2010 deleted the copilot/revert-commit-c6406092 branch May 14, 2026 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants