[PATCH] D48508: [DAGCombiner] eliminate setcc bool math when input is low-bit of some value

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 22 17:17:56 PDT 2018


craig.topper added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:2029
+      N0.getOperand(0).getOpcode() == ISD::SETCC &&
+      N0.getOperand(0).getValueSizeInBits() == 1) {
+    SDValue SetCC = N0.getOperand(0);
----------------
Why not "getValueType() == MVT::i1"?


https://reviews.llvm.org/D48508





More information about the llvm-commits mailing list