[PATCH] D43835: Simplify more cases of logical ops of masked icmps.

Hiroshi Yamauchi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 12 09:55:58 PDT 2018


yamauchi added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineAndOrXor.cpp:450
+  // We are given the canonical form:
+  //   (icmp ne (A & B), 0) & (icmp eq (A & D), E).
+  // where D & E == E.
----------------
davidxl wrote:
> Do you have test case cover this when LSH and RHS are swapped?
> 
> (icmp eq (A &D), E) & (icmp ne (A & B, 0))
Added swapped tests.


Repository:
  rL LLVM

https://reviews.llvm.org/D43835





More information about the llvm-commits mailing list