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

[VPlan] Replace disjoint or with add instead of dropping disjoint. #83821

Merged
merged 4 commits into from
Mar 19, 2024

Commits on Mar 4, 2024

  1. [VPlan] Replace disjoint or with add instead of dropping disjoint.

    Dropping disjoint from an OR may yield incorrect results, as some
    analysis may have converted it to an Add implicitly (e.g. SCEV used
    for dependence analysis). Instead, replace it with an equivalent Add.
    
    This is possible as all users of the disjoint OR only access lanes
    where the operands are disjoint or poison otherwise.
    
    Note that replacing all disjoint ORs with ADDs instead of dropping the
    flags is not strictly necessary. It is only needed for disjoint ORs
    that SCEV treated as ADDs, but those are not tracked.
    fhahn committed Mar 4, 2024
    Configuration menu
    Copy the full SHA
    2a1c471 View commit details
    Browse the repository at this point in the history

Commits on Mar 18, 2024

  1. Configuration menu
    Copy the full SHA
    df5468e View commit details
    Browse the repository at this point in the history
  2. !fixup fix merge, update test.

    fhahn committed Mar 18, 2024
    Configuration menu
    Copy the full SHA
    f4202fc View commit details
    Browse the repository at this point in the history
  3. !fixup fix formatting

    fhahn committed Mar 18, 2024
    Configuration menu
    Copy the full SHA
    dc92c04 View commit details
    Browse the repository at this point in the history