Skip to content

Winch: Add min and max for SIMD floats for x64 with AVX#10271

Merged
saulecabrera merged 1 commit intobytecodealliance:mainfrom
jeffcharles:winch-simd-float-min-max
Feb 25, 2025
Merged

Winch: Add min and max for SIMD floats for x64 with AVX#10271
saulecabrera merged 1 commit intobytecodealliance:mainfrom
jeffcharles:winch-simd-float-min-max

Conversation

@jeffcharles
Copy link
Copy Markdown
Contributor

Part of #8093. Adds support for the following instructions:

  • f32x4.min
  • f64x2.min
  • f32x4.max
  • f64x2.max

I explored putting the float implementations for min and max in the same helper method given how similar they are but the flow is different enough between them that I found it hard to write accurate comments and track what values were in which registers when debugging spec test failures so I've opted to just factor out the NaN canonicalization logic which is identical.

@jeffcharles jeffcharles requested review from a team as code owners February 21, 2025 21:58
@jeffcharles jeffcharles requested review from abrown and dicej and removed request for a team February 21, 2025 21:58
@github-actions github-actions bot added the winch Winch issues or pull requests label Feb 21, 2025
@github-actions
Copy link
Copy Markdown

Subscribe to Label Action

cc @saulecabrera

Details This issue or pull request has been labeled: "winch"

Thus the following users have been cc'd because of the following labels:

  • saulecabrera: winch

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

@abrown abrown requested review from saulecabrera and removed request for abrown and dicej February 24, 2025 21:19
Copy link
Copy Markdown
Member

@saulecabrera saulecabrera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@saulecabrera saulecabrera added this pull request to the merge queue Feb 25, 2025
Merged via the queue into bytecodealliance:main with commit 1f24222 Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

winch Winch issues or pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants