[PATCH] D68949: [DAGCombiner] fold select-of-constants based on sign-bit test

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:8208
+  }
+  return SDValue();
+}
----------------
What about i8 X > -1 ? 0 : C1  or do we canonicalize that to i8 X < 0 ? C1 : 0?


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

https://reviews.llvm.org/D68949