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

[InstCombine] Improve coverage of foldSelectValueEquivalence #88298

Closed
wants to merge 2 commits into from
Closed

[InstCombine] Improve coverage of foldSelectValueEquivalence #88298

wants to merge 2 commits into from

Commits on Jun 6, 2024

  1. [InstCombine] Add tests for expanding foldSelectValueEquivalence; NFC

    @goldsteinn
    goldsteinn committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    689dce0 View commit details
    Browse the repository at this point in the history

  2. [InstCombine] Improve coverage of foldSelectValueEquivalence for co… 

    …nstants

We don't need the `noundef` check if the new simplification is a
constant.

This cleans up regressions from folding multiuse:
    `(icmp eq/ne (sub/xor x, y), 0)` -> `(icmp eq/ne x, y)`.
    @goldsteinn
    goldsteinn committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    a422148 View commit details
    Browse the repository at this point in the history