Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[metadata] Replace uses of MonoType:byref by a getter #60313

Merged
merged 6 commits into from
Oct 14, 2021
Merged

[metadata] Replace uses of MonoType:byref by a getter #60313

merged 6 commits into from
Oct 14, 2021

Commits on Oct 12, 2021

  1. [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.
    @lambdageek
    lambdageek committed Oct 12, 2021
    Configuration menu
    Copy the full SHA
    7257cc8 View commit details
    Browse the repository at this point in the history

  2. Change mono_type_is_byref_internal to take a const argument

    @lambdageek
    lambdageek committed Oct 12, 2021
    Configuration menu
    Copy the full SHA
    4594138 View commit details
    Browse the repository at this point in the history

  3. Missed a couple byref reads

    @lambdageek
    lambdageek committed Oct 12, 2021
    Configuration menu
    Copy the full SHA
    eb2a11e View commit details
    Browse the repository at this point in the history

Commits on Oct 13, 2021

  1. rename mono_type_is_byref_internal -> m_type_is_byref [NFC]

    @lambdageek
    lambdageek committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    c5c4e0f View commit details
    Browse the repository at this point in the history

  2. more missed byrefs

    @lambdageek
    lambdageek committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    d6b938b View commit details
    Browse the repository at this point in the history

  3. revert unintended change

    @lambdageek
    lambdageek committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    76d9194 View commit details
    Browse the repository at this point in the history