-
Notifications
You must be signed in to change notification settings - Fork 5.1k
[WIP] Apx requirements for VM and GC stubs #116806
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Tagging subscribers to this area: @mangod9 |
src/coreclr/inc/regdisp.h
Outdated
PDWORD64 R29; | ||
PDWORD64 R30; | ||
PDWORD64 R31; | ||
//X18 is reserved by OS, in userspace it represents TEB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
//X18 is reserved by OS, in userspace it represents TEB |
DWORD64 R29; | ||
DWORD64 R30; | ||
DWORD64 R31; | ||
struct |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dotnet/dotnet-diag I believe CONTEXT
structure is used by the debugger interfaces. Is this modification going to break them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cannot say for sure at the moment. It depends on the decision if we want the debugger to track these new EGPRs. Once the context structure gets finalized is when we can say with surety about what might get impacted. Thoughts?
DO NOT REVIEW RIGHT NOW
Addresses - #112587