[llvm] [InstCombine] Simplify and/or of icmp eq with op replacement (PR #70335)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 27 00:42:56 PDT 2023


nikic wrote:

> Could you please check whether this PR fixes #68799, #69038

These two are not fixed. They would need an extension like this one: https://github.com/llvm/llvm-project/blob/fb619b3c78b1b6a026b0fec9fccff4c515465803/llvm/lib/Analysis/InstructionSimplify.cpp#L4692-L4716

>, #69050 and #69091?

These are also not fixed, but only because of the recursion limit. I think the way I implemented it may be a bit too strict, as we'll decrement MaxRecurse in simplifyAndOrWithICmpEq() and again in simplifyWithOpReplaced().


https://github.com/llvm/llvm-project/pull/70335


More information about the llvm-commits mailing list