Skip to content

[cDAC] Implement IsMatchingParentFrame for cDAC#127100

Closed
barosiak wants to merge 4 commits intomainfrom
copilot/implement-is-matching-parent-frame
Closed

[cDAC] Implement IsMatchingParentFrame for cDAC#127100
barosiak wants to merge 4 commits intomainfrom
copilot/implement-is-matching-parent-frame

Conversation

@barosiak
Copy link
Copy Markdown
Member

Summary

Implement IsMatchingParentFrame on DacDbiImpl in the cDAC.

Changes

  • DacDbiImpl.cs - IsMatchingParentFrame implementation with direct equality check and legacy validation
  • DacDbiImplTests.cs - new tests covering equal/different frame pointers across architectures

Copilot AI and others added 3 commits April 15, 2026 00:45
…ty check

Replace the legacy-delegation stub with a direct stack pointer equality
comparison, matching the native C++ semantics where
ExInfo::IsUnwoundToTargetParentFrame is simply sfParent == sfToCheck.

Include #if DEBUG legacy cross-validation following the IsValueType pattern.
Add [cDAC] annotation to the native C++ implementation.
Add unit tests covering equal, different, zero, and max value frame pointers.

Agent-Logs-Url: https://github.com/dotnet/runtime/sessions/bdff63a2-7448-46cb-8c0e-ffa04ae69ec8

Co-authored-by: barosiak <76071368+barosiak@users.noreply.github.com>
…data points

Agent-Logs-Url: https://github.com/dotnet/runtime/sessions/bdff63a2-7448-46cb-8c0e-ffa04ae69ec8

Co-authored-by: barosiak <76071368+barosiak@users.noreply.github.com>
@barosiak barosiak self-assigned this Apr 18, 2026
Copilot AI review requested due to automatic review settings April 18, 2026 00:13
@barosiak barosiak marked this pull request as ready for review April 18, 2026 00:15
@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.

@rcj1
Copy link
Copy Markdown
Contributor

rcj1 commented Apr 18, 2026

Hey, I just put up a PR such that we won’t need this API: #127099

@rcj1 rcj1 closed this Apr 18, 2026
@barosiak barosiak deleted the copilot/implement-is-matching-parent-frame branch April 18, 2026 00:19
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