Skip to content

Conversation

@kg
Copy link
Member

@kg kg commented Jan 11, 2023

These new ADD_MUL opcodes show up a lot so it's worth implementing them.

Fp relop branches were supposed to be working already, but I failed to wire up the helper code properly so they weren't being generated.

Also added MINT_INTRINS_GET_HASHCODE and MINT_INTRINS_RUNTIMEHELPERS_OBJECT_HAS_COMPONENT_SIZE.

@kg kg added the arch-wasm WebAssembly architecture label Jan 11, 2023
@kg kg requested review from lewing and pavelsavara as code owners January 11, 2023 22:33
@ghost ghost assigned kg Jan 11, 2023
@ghost
Copy link

ghost commented Jan 11, 2023

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

These new ADD_MUL opcodes show up a lot so it's worth implementing them.

Fp relop branches were supposed to be working already, but I failed to wire up the helper code properly so they weren't being generated.

Author: kg
Assignees: -
Labels:

arch-wasm

Milestone: -

@kg kg requested review from BrzVlad and vargaz as code owners January 11, 2023 23:17
@kg kg changed the title [wasm] Implement the ADD_MUL indirect loads and fp relop branches in the jiterpreter [wasm] Implement more jiterpreter opcodes Jan 11, 2023
@kg kg merged commit da2c147 into dotnet:main Jan 12, 2023
kg pushed a commit that referenced this pull request Feb 7, 2023
#80520 added support in the Jiterpreter for the intrinsic that underlies RuntimeHelpers.GetHashCode. This PR adds similar support for the intrinsic that underlies RuntimeHelpers.TryGetHashCode.

For reference RuntimeHelpers.TryGetHashCode was added in #80059. It is currently only used in ConditionalWeakTable.TryGetValue.
@ghost ghost locked as resolved and limited conversation to collaborators Feb 11, 2023
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.

2 participants