[PATCH] D118037: [DAGCombine] Consider SETULT as carry flag

[Paweł Bylica via Phabricator via llvm-commits]

chfast added a comment.

What do you think about this idea in general? Can we assume or check if SETULT actually affects CL flag on given target?



================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:2609
+    if (CC == ISD::SETULT)
+      return V;
+    return {};
----------------
RKSimon wrote:
> Do we need any getBooleanContents checks here?
I'm not sure. But I can pass it though the `getBooleanContents` check below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118037