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

Mon Nov 21 06:02:47 PST 2022

foad added a comment.

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?

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

https://reviews.llvm.org/D138401