[llvm] [ValueTracking] Extend LHS/RHS with matching operand to work without constants. (PR #85557)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Mar 17 09:21:13 PDT 2024
goldsteinn wrote:
> Yeah, I can't say I'm particularly fond of the direction.
>
> From the test diffs, a fold we're missing is this: https://alive2.llvm.org/ce/z/-njJr8 Particularly profitable if the new comparison folds later, but also seems generally beneficial.
Would think its more principled to more generally handle the implication between conditions rather than add a transform for each possible case.
https://github.com/llvm/llvm-project/pull/85557
More information about the llvm-commits
mailing list