[llvm] [InstCombine] Extend bitmask mul combine to handle independent operands (PR #142503)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 26 23:54:49 PDT 2025


dtcxzyw wrote:

> Looking into the details -- turning out to be a larger refactor than I had thought. How do you feel about providing the functionality using the current implementation, then refactoring into reassociateBooleanAndOr as an extension?

You don't need to refactor `reassociateBooleanAndOr`. It also handles boolean logical and/or so it is not suitable for non-boolean or instructions. I guess nikic meant to add a new helper like `reassociateDisjointOr`.


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


More information about the llvm-commits mailing list