Skip to content

[wasm][jiterp] Implement i64 jiterp shuffle and simplify logic #115344

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 24 commits into
base: main
Choose a base branch
from
Draft

[wasm][jiterp] Implement i64 jiterp shuffle and simplify logic #115344

wants to merge 24 commits into from
+104 −71

Conversation

lewing
Copy link
Member

@lewing lewing commented May 6, 2025 •
edited
Loading

Implement i64 jiterp shuffle for both constant and variable indices. Unify the shuffle path handling in the process.

@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 6, 2025
@lewing lewing closed this May 6, 2025
@lewing lewing reopened this May 6, 2025
@lewing lewing requested a review from kg May 6, 2025 22:49
@lewing lewing added this to the 10.0.0 milestone May 6, 2025
@lewing lewing added area-Codegen-Interpreter-mono and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels May 6, 2025
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

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

This was referenced May 7, 2025
Open
Open
Closed
@build-analysis build-analysis bot mentioned this pull request May 7, 2025
Closed
@lewing lewing changed the title [wasm][jiterp] Implement i8 jiterp shuffle and simplify logic [wasm][jiterp] Implement i64 jiterp shuffle and simplify logic May 7, 2025
kg
kg reviewed May 7, 2025
View reviewed changes
kg
kg reviewed May 7, 2025
View reviewed changes
kg
kg reviewed May 7, 2025
View reviewed changes
kg
kg reviewed May 7, 2025
View reviewed changes
kg
kg reviewed May 7, 2025
View reviewed changes
kg
kg reviewed May 7, 2025
View reviewed changes
kg
kg reviewed May 7, 2025
View reviewed changes
kg
kg reviewed May 7, 2025
View reviewed changes
This was referenced May 7, 2025
Closed
Open
@lewing lewing marked this pull request as ready for review May 7, 2025 20:28
@lewing lewing requested a review from pavelsavara as a code owner May 7, 2025 20:28
@lewing lewing requested a review from radekdoulik May 7, 2025 20:28
kg
kg reviewed May 8, 2025
View reviewed changes
kg
kg reviewed May 8, 2025
View reviewed changes
@lewing lewing mentioned this pull request May 8, 2025
Open
12 tasks
lewing
lewing commented May 8, 2025
View reviewed changes
@lewing lewing marked this pull request as draft September 9, 2025 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kg kg kg left review comments

@pavelsavara pavelsavara Awaiting requested review from pavelsavara pavelsavara is a code owner

@radekdoulik radekdoulik Awaiting requested review from radekdoulik

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

@lewing lewing

Labels
area-Codegen-Interpreter-mono
Projects
None yet
Milestone
10.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@lewing @kg