[PATCH] D31944: [DAGCombiner] add (sext i1 X), 1 --> zext (not i1 X)

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:1885
+        N0.getOperand(0).getScalarValueSizeInBits() == 1 &&
+        isOneConstantOrOneSplatConstant(N1) && !LegalOperations) {
+      SDValue X = N0.getOperand(0);
----------------
!LegalOperations check first? Also why not check if the xor/zext are legal?


https://reviews.llvm.org/D31944