Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x64: Add native lowering for scalar fma #4539

Merged
merged 1 commit into from
Aug 11, 2022

Conversation

afonso360
Copy link
Contributor

@afonso360 afonso360 commented Jul 27, 2022

👋 Hey

This PR adds a lowering for scalar fma for x64. Additionally we enable this operation in the CLIF Fuzzer.

cc: @abrown

@afonso360 afonso360 changed the title x64: Add lowering for scalar fma x64: Add native lowering for scalar fma Jul 27, 2022
@github-actions github-actions bot added cranelift Issues related to the Cranelift code generator cranelift:area:x64 Issues related to x64 codegen labels Jul 27, 2022
@jameysharp
Copy link
Contributor

This PR has unfortunately gone stale, but I'd like to not lose track of it. @afonso360, could you give it a rebase?

@github-actions github-actions bot added the isle Related to the ISLE domain-specific language label Aug 11, 2022
@github-actions
Copy link

Subscribe to Label Action

cc @cfallin, @fitzgen

This issue or pull request has been labeled: "cranelift", "cranelift:area:x64", "isle"

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

  • cfallin: isle
  • fitzgen: isle

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

Learn more.

@afonso360
Copy link
Contributor Author

afonso360 commented Aug 11, 2022

Changes with the last rebase:

  • We now query the use_fma flag to lower FMA extension instructions
  • Added some compile tests for both the libcall and inst case

@afonso360
Copy link
Contributor Author

CI failures seem spurious

Copy link
Member

@elliottt elliottt left a comment

Choose a reason for hiding this comment

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

This looks great to me! Thanks for adding all the filetest cases!

@jameysharp
Copy link
Contributor

CI failures are because today's release of Rust 1.63 reports a new warning. #4691 is merged fixing that, so please rebase and then hopefully CI will pass.

Use `vfmadd213{ss,sd}` for these lowerings.
@jameysharp jameysharp enabled auto-merge (squash) August 11, 2022 22:48
@jameysharp jameysharp merged commit 3ea1813 into bytecodealliance:main Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cranelift:area:x64 Issues related to x64 codegen cranelift Issues related to the Cranelift code generator isle Related to the ISLE domain-specific language
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants