Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[wasm][aot] Fix op_OnesComplement on wasm and enable ceil/floor #115151

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

Merged
merged 2 commits into from
May 5, 2025

Conversation

lewing
Copy link
Member

@lewing lewing commented Apr 29, 2025

No description provided.

@lewing lewing changed the title [wasm][aot] Fill dome missing intrinsics cases [wasm][aot] Fill in some missing intrinsics cases Apr 29, 2025
@lewing lewing requested a review from radekdoulik April 29, 2025 14:11
@lewing lewing changed the title [wasm][aot] Fill in some missing intrinsics cases [wasm][aot] Fix op_OnesComplement on wasm and enable ceil/floor May 4, 2025
@lewing lewing marked this pull request as ready for review May 4, 2025 20:20
@Copilot Copilot AI review requested due to automatic review settings May 4, 2025 20:20
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the handling of the op_OnesComplement intrinsic on wasm and expands the platform support for ceil/floor SIMD operations.

  • Adds new cases for SN_op_UnaryNegation and SN_op_OnesComplement in the SIMD intrinsic switch.
  • Expands the conditional compilation for ceil/floor operations to support TARGET_ARM and TARGET_WASM.

@lewing lewing requested a review from kg May 4, 2025 20:20
@lewing
Copy link
Member Author

lewing commented May 5, 2025

/ba-g timeouts are not related

@lewing lewing merged commit 5586307 into dotnet:main May 5, 2025
76 of 84 checks passed
@lewing lewing deleted the fix-aot-intrins branch May 5, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants