[llvm] [InstCombine] Make the `(icmp eq/ne (and X, Y), X)` canonicalization work for non-const operands (PR #84688)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Sat May 25 00:11:21 PDT 2024
https://github.com/nikic approved this pull request.
The first two commits LGTM. I don't think we need the last one -- the cases it improves look very artificial to me. If you do want to have that one, I think it's best to make a separate PR so @dtcxzyw can check whether the problematic patterns occur in the wild. But in the meantime I don't mind regressing them.
https://github.com/llvm/llvm-project/pull/84688
More information about the llvm-commits
mailing list