[PATCH] D138401: [DAGCombiner] fold or (xor x, y),? patterns

Mon Nov 21 23:00:29 PST 2022

bcl5980 added a comment.

In D138401#3940984 <https://reviews.llvm.org/D138401#3940984>, @foad wrote:

> In D138401#3940345 <https://reviews.llvm.org/D138401#3940345>, @bcl5980 wrote:
>
>> In D138401#3940326 <https://reviews.llvm.org/D138401#3940326>, @foad wrote:
>>
>>> Does instcombine do this at the IR level? Is there a need to do it in selectiondag too?
>>
>> Yeah, instsimplify do this. But I find some corner cases in selectiondag can trigger this code because of some logical op generated by SelectionDAG itself. So, I add all similar code in this change.
>
> Can you add a test that shows that happening?

If you think the test is OK I can precommit the test.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138401/new/

https://reviews.llvm.org/D138401