Skip to content

[wasm][interp] Filter out overmatched shift#114492

Merged
lewing merged 5 commits into
dotnet:mainfrom
lewing:fix-shift
Apr 10, 2025
Merged

[wasm][interp] Filter out overmatched shift#114492
lewing merged 5 commits into
dotnet:mainfrom
lewing:fix-shift

Conversation

@lewing

@lewing lewing commented Apr 10, 2025

Copy link
Copy Markdown
Member

@lewing lewing marked this pull request as ready for review April 10, 2025 15:17
Copilot AI review requested due to automatic review settings April 10, 2025 15:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/mono/mono/mini/interp/transform-simd.c:1089

  • Ensure that unit tests cover both the HOST_BROWSER and non-HOST_BROWSER branches to verify the correct behavior of the SN_get_IsSupported/IsHardwareAccelerated handling.
#if HOST_BROWSER

Comment thread src/mono/mono/mini/interp/transform-simd.c
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @BrzVlad, @kotlarmilos
See info in area-owners.md if you want to be subscribed.

@lewing lewing requested a review from javiercn April 10, 2025 15:18
@lewing

lewing commented Apr 10, 2025

Copy link
Copy Markdown
Member Author

/ba-g timeouts passed pertinent tests before timing out and are unrelated

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants