Skip to content

[DO NOT MERGE] - #131490

Draft
rcj1 with Copilot wants to merge 5 commits into
mainfrom
copilot/add-new-dacdbi-apis
Draft

[DO NOT MERGE]#131490
rcj1 with Copilot wants to merge 5 commits into
mainfrom
copilot/add-new-dacdbi-apis

Conversation

Copilot AI commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Adds GetReadWriteMetadataSize and FillReadWriteMetadata from #131027 without removing existing metadata infrastructure or DacDbi APIs.

  • Native DAC
    • Serializes writable metadata into a contiguous ECMA-335 image.
    • Implements size-query and caller-provided-buffer APIs.
  • cDAC
    • Exposes writable metadata serialization through IEcmaMetadata.
    • Implements matching managed DacDbi entry points.

Note

This description was generated by GitHub Copilot.

Copilot AI and others added 3 commits July 28, 2026 19:51
Co-authored-by: rcj1 <77995559+rcj1@users.noreply.github.com>
Co-authored-by: rcj1 <77995559+rcj1@users.noreply.github.com>
Co-authored-by: rcj1 <77995559+rcj1@users.noreply.github.com>
Copilot AI self-assigned this Jul 28, 2026
Copilot AI review requested due to automatic review settings July 28, 2026 20:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot finished work on behalf of rcj1 July 28, 2026 20:11
Copilot AI requested a review from rcj1 July 28, 2026 20:11
@rcj1 rcj1 added NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels Jul 28, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 5 pipeline(s).
11 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

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

Co-authored-by: rcj1 <77995559+rcj1@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 28, 2026 22:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Co-authored-by: rcj1 <77995559+rcj1@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 28, 2026 22:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot finished work on behalf of rcj1 July 28, 2026 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-Diagnostics-coreclr NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants