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

Be consistent about caching behavior of function resolutions through IDynamicInterfaceCastable #107999

Open
davidwrighton opened this issue Sep 18, 2024 · 2 comments
Open

Be consistent about caching behavior of function resolutions through IDynamicInterfaceCastable #107999

davidwrighton opened this issue Sep 18, 2024 · 2 comments
Labels
area-NativeAOT-coreclr
Milestone
10.0.0

Comments

@davidwrighton
Copy link
Member

Based on discussion in #106843 (comment) we need to validate the caching behavior here and not cause the runtimes to do variable amounts of work here.
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Sep 18, 2024
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Sep 18, 2024
@davidwrighton
Copy link
Member Author

We need to write some tests to validate that my thoughts on the caching behavior are correct, and then if I am right, we should be able to remove the special casing in NativeAOT cached interface dispatch path.

@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

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

@davidwrighton davidwrighton mentioned this issue Sep 18, 2024
Merged
@jkotas jkotas removed the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Sep 18, 2024
@agocke agocke added this to the 10.0.0 milestone Sep 18, 2024
@agocke agocke removed the untriaged New issue has not been triaged by the area owner label Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-NativeAOT-coreclr
Projects
AppModel
Status: No status
Milestone
10.0.0
Development

No branches or pull requests
3 participants
@agocke @jkotas @davidwrighton