[wasm][interp] Jiterp packedsimd shifts by lewing · Pull Request #114669 · dotnet/runtime · GitHub
Skip to content

[wasm][interp] Jiterp packedsimd shifts#114669

Merged
lewing merged 7 commits into
dotnet:mainfrom
lewing:jiterp-packedsimd-shifts
Apr 15, 2025
Merged

[wasm][interp] Jiterp packedsimd shifts#114669
lewing merged 7 commits into
dotnet:mainfrom
lewing:jiterp-packedsimd-shifts

Conversation

@lewing

@lewing lewing commented Apr 15, 2025

Copy link
Copy Markdown
Member

Fill in some of the Jiterp tables with the PackedSimd intrinsics so that both aliased and direct PackedSimd calls in the interpreter also benefit from the Jiterpreter. Also reenable the aliased opcodes now that they will share the jiterpreter optimizations.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

@lewing lewing requested a review from kg April 15, 2025 03:04
@lewing

lewing commented Apr 15, 2025

Copy link
Copy Markdown
Member Author

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 4 out of 4 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/mono/browser/runtime/jiterpreter-trace-generator.ts:3691

  • Consider adding targeted tests to validate the early bitmask branch logic in emit_simd_2 to ensure that both the bitmask and fallback paths are exercised.
const bitmask = bitmaskTable[index];

Comment thread src/mono/mono/mini/interp/transform-simd.c
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