-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[Wasm RyuJit] Ind write barriers, direct helper calls, more #124113
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
Conversation
AndyAyersMS
commented
Feb 6, 2026
- implement codegen for write barrier helper calls
- support direct calls to helpers
- decorate call targets with their symbolic names in dumps/disasm
* implement codegen for write barrier helper calls * support direct calls to helpers * decorate call targets with their symbolic names in dumps/disasm
|
fyi @dotnet/jit-contrib |
There was a problem hiding this 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 extends the CoreCLR WASM JIT backend to better support runtime helper calls and GC write barriers, and improves dump/disasm readability by annotating call targets with symbolic names.
Changes:
- Add disasm/dump decoration for call targets in
emitwasm.cppusing recorded method handles. - Implement WASM
GT_STOREINDwrite barrier codegen by routing togenGCWriteBarrier. - Enable direct helper-call emission in WASM codegen and adjust
codegencommon.cppTARGET_WASMguards to keep non-WASM-only code excluded while sharing write-barrier helpers.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/coreclr/jit/emitwasm.cpp | Prints call target method names in verbose dumps/disasm when debug info is present. |
| src/coreclr/jit/codegenwasm.cpp | Adds write-barrier handling for STOREIND and emits helper calls via genEmitHelperCall. |
| src/coreclr/jit/codegencommon.cpp | Refactors TARGET_WASM preprocessor guards to share write-barrier helper selection while excluding non-WASM-only sections. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
|
Tagging subscribers to 'arch-wasm': @lewing, @pavelsavara |
…24113) * implement codegen for write barrier helper calls * support direct calls to helpers * decorate call targets with their symbolic names in dumps/disasm