[PATCH] D71816: [DAGCombiner] Add combine for (not (strict_fsetcc)) to create a strict_fsetcc with the opposite condition.

[Pengfei Wang via Phabricator via llvm-commits]

pengfei added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:7073
+        if (N0.hasOneUse()) {
+          // FIXME Can we handle multiples uses? Could we token factor the chain
+          // results from the new/old setcc?
----------------
Why we need consider cases more than one use? Isn't xor + setcc better that 2 setccs?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71816