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.

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

add missing invoke edge for nospecialize targets #51036

Merged
merged 1 commit into from
Aug 26, 2023
Merged

add missing invoke edge for nospecialize targets #51036

merged 1 commit into from
Aug 26, 2023

Commits on Aug 24, 2023

  1. add missing invoke edge for nospecialize targets

    We need 2 edges: one for the lookup (which uses the call signature) and
    one for the invoke (which uses the invoke signature). It is hard to make
    a small example for this, but the test case demonstrated this issue,
    particularly if inspected by `SnoopCompile.@snoopr`.
    
    Additionally, we can do some easy optimizations on the invoke
    invalidation, since in most cases we know from subtyping transativity
    that it is only invalid if the method callee target is actually deleted,
    and otherwise it cannot ever be partially replaced.
    
    Fixes: #50091
    vtjnash committed Aug 24, 2023
    Configuration menu
    Copy the full SHA
    c6209b9 View commit details
    Browse the repository at this point in the history