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

Improve fgValueNumberBlockAssignment #64110

Merged
merged 1 commit into from
Feb 3, 2022

Commits on Jan 28, 2022

  1. Refactor fgValueNumberBlockAssignment

    "fgValueNumberBlockAssignment" had a very interesting oddity:
    it re-VNs the source tree of the assignment. This is unnecessary,
    and is in fact a pessimization, as it means we will fail to VN
    assignments from sources it does not understand. This change
    fixes that, bringing along some positive diffs from numbering
    stores from field indirections.
    
    "fgValueNumberBlockAssignment" also needs to maintaint the
    invariant that a location's VN will always match its type.
    It was failing to do that in cases where the assignment's
    source was not local. This change fixes that.
    
    Finally, this change unifies the code common to numbering
    "CopyBlk" and "InitBlk" cases. There is no need for them to
    be different.
    SingleAccretion committed Jan 28, 2022
    Configuration menu
    Copy the full SHA
    4601ecc View commit details
    Browse the repository at this point in the history