[PATCH] D101807: [InstCombine] Fold select x_inv, true, (select y, x, false)

[Juneyoung Lee via Phabricator via llvm-commits]

aqjune added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp:2776
     }
+    // select c, true, (select a, b, false)  -> select c, true, a
+    //   if c = false implies that b = true
----------------
nikic wrote:
> Shouldn't we also handle `select (select a, b, false), true, c -> select a, true, c` here? It seems odd to handle 3 out of 4 patterns.
Fine, since this is just a repetition of the current change I included it in this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101807