Skip to content

Update Microsoft.Diagnostics.Runtime to 4.0.0-beta.26217.1#127088

Open
max-charlamb wants to merge 2 commits intomainfrom
dev/max-charlamb/update-clrmd-cross-bitness
Open

Update Microsoft.Diagnostics.Runtime to 4.0.0-beta.26217.1#127088
max-charlamb wants to merge 2 commits intomainfrom
dev/max-charlamb/update-clrmd-cross-bitness

Conversation

@max-charlamb
Copy link
Copy Markdown
Member

Update Microsoft.Diagnostics.Runtime (ClrMD) from 4.0.0-beta.26210.1 to 4.0.0-beta.26217.1.

This version includes cross-bitness dump reading support, which replaces all host-dependent IntPtr.Size usages with the target's PointerSize.

Changes in ClrMD 4.0.0-beta.26217.1

  • Replace IntPtr.Size with target PointerSize across 14 files
  • Fix bug in CommonMemoryReader.ReadPointer wrong overload binding
  • Remove ambiguous SpanExtensions.AsPointer overloads
  • Remove SigParser default constructors
  • Clean up dead code in MetadataImport and SigParser

This version includes cross-bitness dump reading support (microsoft/clrmd#1421),
allowing correct reading of dumps from different architectures (e.g., ARM64
dumps from an x86 host).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 17, 2026 17:51
@github-actions github-actions bot added the area-codeflow for labeling automated codeflow label Apr 17, 2026
Copy link
Copy Markdown
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

Updates the repo’s pinned ClrMD dependency version to pick up the newer Microsoft.Diagnostics.Runtime beta that includes cross-bitness dump reading support (upstream changes replace host IntPtr.Size assumptions with target PointerSize).

Changes:

  • Bump MicrosoftDiagnosticsRuntimeVersion from 4.0.0-beta.26210.1 to 4.0.0-beta.26217.1.

The xplat dump tests were hitting OutOfMemoryException when the Helix SDK
tried to zip all platforms' dumps into a single MemoryStream-backed
ZipArchive, exceeding the 2GB byte[] limit.

Fix by splitting payloads:
- Test DLLs: sent as a correlation payload (shared, uploaded once)
- Dumps: sent as per-work-item payloads (one platform per work item)

This avoids the single large zip and reduces redundant uploads since test
DLLs are no longer duplicated across work items.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-codeflow for labeling automated codeflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants