[PATCH] D70913: [FPEnv][SelectionDAG] Relax chain requirements

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 11:11:05 PST 2019


craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:3181
+        SDNode *NS = NodeStack[i].getNode();
+        for (auto UI = NS->use_begin(), UE = NS->use_end(); UI != UE; ++UI)
+          if (UI.getUse().getValueType() != MVT::Other)
----------------
Can we break this loop when NonChainUses is greater than 1? No need to keep scanning.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70913





More information about the llvm-commits mailing list