Skip to content

[DAC] properly 'dacize' async variant method sigs#120946

Merged
max-charlamb merged 1 commit intodotnet:mainfrom
max-charlamb:dacize-async-sig
Oct 22, 2025
Merged

[DAC] properly 'dacize' async variant method sigs#120946
max-charlamb merged 1 commit intodotnet:mainfrom
max-charlamb:dacize-async-sig

Conversation

@max-charlamb
Copy link
Member

Based off of existing StoredSigMethodDesc::GetStoredMethodSig

@max-charlamb max-charlamb requested a review from tommcdon October 21, 2025 16:44
Copilot AI review requested due to automatic review settings October 21, 2025 16:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds proper DAC (Data Access Component) support for async variant method signatures by ensuring the signature data is correctly marshaled when accessed from out-of-process debugging scenarios.

Key Changes:

  • Adds DAC-specific handling in MethodDesc::GetSig() for async variant methods to properly instantiate signature data in the debugger's address space

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @steveisok, @dotnet/dotnet-diag
See info in area-owners.md if you want to be subscribed.

Copy link
Member

@tommcdon tommcdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@max-charlamb max-charlamb merged commit 3dc26a3 into dotnet:main Oct 22, 2025
100 checks passed
@max-charlamb max-charlamb deleted the dacize-async-sig branch October 22, 2025 20:42
@github-actions github-actions bot locked and limited conversation to collaborators Nov 22, 2025
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.

3 participants