-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[metadata] Replace uses of MonoType:byref by a getter #60313
Conversation
Call `mono_type_is_byref_internal` instead of accessing the byref bit directly. No functional change. (I hope). This is in preparation for exploring alternative representations of ref types.
Tagging subscribers to this area: Issue DetailsCall No functional change. (I hope). This is in preparation for exploring alternative representations of ref types.
|
Tagging subscribers to this area: Issue DetailsCall No functional change. (I hope). This is in preparation for exploring alternative representations of ref types.
|
Can it have a shorter name like m_type_is_byref () ? |
@vargaz changed to a shorter name. How's it look? |
* [metadata] Replace uses of MonoType:byref by a getter Call `mono_type_is_byref_internal` instead of accessing the byref bit directly. No functional change. (I hope). This is in preparation for exploring alternative representations of ref types. * Change mono_type_is_byref_internal to take a const argument * Missed a couple byref reads * rename mono_type_is_byref_internal -> m_type_is_byref [NFC] * more missed byrefs * revert unintended change
Those were missed in dotnet#60313 and caused a build break on s390x. I fixed all other occurences I could find too.
Those were missed in #60313 and caused a build break on s390x. I fixed all other occurences I could find too.
Call
m_type_is_byref
instead of accessing the byref bit directly.No functional change. (I hope).
This is in preparation for exploring alternative representations of ref types.