-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[Mono] Fix unsafe accessor related issue for full AOT #91216
Conversation
/azp run runtime-extra-platforms |
Azure Pipelines successfully started running 1 pipeline(s). |
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.
please check the wrapper subtype also when skipping the assertion
/backport to release/8.0 |
Started backporting to release/8.0: https://github.com/dotnet/runtime/actions/runs/6012851339 |
/azp run runtime-extra-platforms |
Azure Pipelines successfully started running 1 pipeline(s). |
Is there test somewhere? |
This was actually exposed via certain tests in our CI running via extra-platforms lane (unfortunately hardware limitation don't allow to run on per PR basis). |
|
Local testing:
|
Fixes #91066
This PR fixed two issues:
WRAPPER_SUBTYPE_UNSAFE_ACCESSOR
member_name
is not NULL