Skip to content

Adding _tls_index global pointer to DACVARs#125318

Merged
rcj1 merged 3 commits intodotnet:mainfrom
rcj1:add-cdac-pointers
Mar 10, 2026
Merged

Adding _tls_index global pointer to DACVARs#125318
rcj1 merged 3 commits intodotnet:mainfrom
rcj1:add-cdac-pointers

Conversation

@rcj1
Copy link
Contributor

@rcj1 rcj1 commented Mar 9, 2026

Add this so that _tls_index is picked up in mini- and heap-dump scenarios.

@dotnet-policy-service
Copy link
Contributor

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

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

Adds a DACVAR entry for the CRT-provided _tls_index so it can be located in mini-/heap-dump debugging scenarios.

Changes:

  • Registers ::_tls_index in dacvars.h via DEFINE_DACVAR(...).

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 9, 2026 04:24
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

@rcj1 rcj1 merged commit 3b580ea into dotnet:main Mar 10, 2026
105 checks passed
@rcj1 rcj1 deleted the add-cdac-pointers branch March 10, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants