[PATCH] D61511: [DAGCombiner] Limit number of nodes explored as store candidates.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 6 12:38:21 PDT 2019


fhahn added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:14845
+    for (auto I = RootNode->use_begin(), E = RootNode->use_end();
+         I != E && NumNodesExplored < 1024; ++I, NumNodesExplored++)
       if (I.getOperandNo() == 0 && isa<LoadSDNode>(*I)) // walk down chain
----------------
niravd wrote:
> nit: use prefix increment to be consistent with other uses.
Done,thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61511





More information about the llvm-commits mailing list