[llvm] [InstCombine] Improve coverage of `foldSelectValueEquivalence` for non-constants (PR #94719)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 12 10:33:30 PDT 2024


dtcxzyw wrote:

> I don't think this is worth the complication. The only part of this I would consider is the `NewOp == V` case, though I'm having some trouble convincing myself that this really can't lead to a combine loop.

I agree.


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


More information about the llvm-commits mailing list