Skip to content
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

[mono][interpreter] Fix performance of new span helpers #76326

Closed
lewing opened this issue Sep 28, 2022 · 3 comments
Closed

[mono][interpreter] Fix performance of new span helpers #76326

lewing opened this issue Sep 28, 2022 · 3 comments
Assignees
@BrzVlad
Labels
area-Codegen-Interpreter-mono
Milestone
8.0.0

Comments

@lewing
Copy link
Member

lewing commented Sep 28, 2022

#73768 regressed Mono. It was "reverted" in #75917 to work around the regression but largely fixed for AOT in #76033 but we cannot currently remove the workaround because the interpreter is still impacted.
@lewing lewing added this to the 8.0.0 milestone Sep 28, 2022
@ghost
Copy link

ghost commented Sep 28, 2022

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

Issue Details

#73768 regressed Mono. It was "reverted" in #75917 to work around the regression but largely fixed for AOT in #76033 but we cannot currently remove the workaround because the interpreter is still impacted.

Author: lewing
Assignees: -
Labels:

area-Codegen-Interpreter-mono
Milestone: 8.0.0

@lewing
Copy link
Member Author

lewing commented Sep 28, 2022

cc @adamsitnik

@lewing
Copy link
Member Author

lewing commented Dec 20, 2022

fixed in #79215

@lewing lewing closed this as completed Dec 20, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jan 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@BrzVlad BrzVlad

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

No branches or pull requests
2 participants
@lewing @BrzVlad