Skip to content

Conversation

@kg
Copy link
Member

@kg kg commented Jan 5, 2023

The current jiterpreter heuristic that rules out trace insertion points doesn't handle branches very accurately. This PR improves its understanding of branches a bit more, raising the number of traces successfully compiled and (in my testing) slightly improving its accuracy as well.

@kg kg added the arch-wasm WebAssembly architecture label Jan 5, 2023
@ghost
Copy link

ghost commented Jan 5, 2023

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

Issue Details

The current jiterpreter heuristic that rules out trace insertion points doesn't handle branches very accurately. This PR improves its understanding of branches a bit more, raising the number of traces successfully compiled and (in my testing) slightly improving its accuracy as well.

Author: kg
Assignees: -
Labels:

arch-wasm

Milestone: -

@ghost ghost assigned kg Jan 5, 2023
@ghost
Copy link

ghost commented Jan 5, 2023

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

Issue Details

The current jiterpreter heuristic that rules out trace insertion points doesn't handle branches very accurately. This PR improves its understanding of branches a bit more, raising the number of traces successfully compiled and (in my testing) slightly improving its accuracy as well.

Author: kg
Assignees: -
Labels:

arch-wasm, area-Codegen-Interpreter-mono

Milestone: -

@kg kg merged commit 027e19b into dotnet:main Jan 6, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Feb 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants