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

some ConstValue refactoring #115764

Merged
merged 7 commits into from
Sep 14, 2023
Merged

some ConstValue refactoring #115764

merged 7 commits into from
Sep 14, 2023

Commits on Sep 14, 2023

  1. use AllocId instead of Allocation in ConstValue::ByRef

    @RalfJung
    RalfJung committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    551f481 View commit details
    Browse the repository at this point in the history

  2. cleanup op_to_const a bit; rename ConstValue::ByRef → Indirect

    @RalfJung
    RalfJung committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    0f8908d View commit details
    Browse the repository at this point in the history

  3. make it more clear which functions create fresh AllocId

    @RalfJung
    RalfJung committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    430c386 View commit details
    Browse the repository at this point in the history

  4. don't force all slice-typed ConstValue to be ConstValue::Slice

    @RalfJung
    RalfJung committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    7aa44ee View commit details
    Browse the repository at this point in the history

  5. bless all

    @RalfJung
    RalfJung committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    0a6e263 View commit details
    Browse the repository at this point in the history

  6. fix clippy (and MIR printing) handling of ConstValue::Indirect slices

    @RalfJung
    RalfJung committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    b18c0a8 View commit details
    Browse the repository at this point in the history

  7. add regression test for something we fixed

    @RalfJung
    RalfJung committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    04a4df5 View commit details
    Browse the repository at this point in the history