[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