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

sem: replace addr calls with nkAddr #1465

Merged
merged 4 commits into from
Oct 19, 2024
Merged

sem: replace addr calls with nkAddr #1465

merged 4 commits into from
Oct 19, 2024

Commits on Oct 18, 2024

  1. sem: turn addr call into nkAddr node 

    This ensures that all address-of operations are represented by `nkAddr`
trees in typed AST. Previously, the calls stayed in the AST as `mAddr`
calls, requiring analysis to also test for those (which not all
analysis did consistently).
    @zerbina
    zerbina committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    f157d1c View commit details
    Browse the repository at this point in the history

  2. compiler: clean up after nkAddr change 

    There are no more `mAddr` calls in typed AST.
    @zerbina
    zerbina committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    236713b View commit details
    Browse the repository at this point in the history

  3. tests: add a test for the fixed effect tracking bug 

    Effect tracking not special casing the `mAddr` call led to `addr x`
being considered an indirect call of `x`. With `mAddr` calls gone, this
is now fixed.
    @zerbina
    zerbina committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    60e9aab View commit details
    Browse the repository at this point in the history

  4. tests: update the tmacrostmt test

    @zerbina
    zerbina committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    d1dda9d View commit details
    Browse the repository at this point in the history