From: Evan Cheng <evan.cheng@apple.com>
Date: Thu, 25 May 2006 20:16:55 +0000 (+0000)
Subject: Can't trust NodeDepth when checking for possibility of load folding creating
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=553ef1be94dffdedbc6e1fce9de91f77e5f12b95;p=oota-llvm.git

Can't trust NodeDepth when checking for possibility of load folding creating
a cycle. This increase the search space and will increase compile time (in
practice it appears to be small, e.g. 176.gcc goes from 62 sec to 65 sec)
that will be addressed later.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@28476 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/utils/TableGen/DAGISelEmitter.cpp b/utils/TableGen/DAGISelEmitter.cpp
index 8f2744f012b..4a399cdfc03 100644
--- a/utils/TableGen/DAGISelEmitter.cpp
+++ b/utils/TableGen/DAGISelEmitter.cpp
@@ -3481,13 +3481,11 @@ void DAGISelEmitter::run(std::ostream &OS) {
   OS << "  if (found || !Visited.insert(Use).second) return;\n";
   OS << "  for (unsigned i = 0, e = Use->getNumOperands(); i != e; ++i) {\n";
   OS << "    SDNode *N = Use->getOperand(i).Val;\n";
-  OS << "    if (N->getNodeDepth() >= Def->getNodeDepth()) {\n";
-  OS << "      if (N != Def) {\n";
-  OS << "        findNonImmUse(N, Def, found, Visited);\n";
-  OS << "      } else {\n";
-  OS << "        found = true;\n";
-  OS << "        break;\n";
-  OS << "      }\n";
+  OS << "    if (N != Def) {\n";
+  OS << "      findNonImmUse(N, Def, found, Visited);\n";
+  OS << "    } else {\n";
+  OS << "      found = true;\n";
+  OS << "      break;\n";
   OS << "    }\n";
   OS << "  }\n";
   OS << "}\n";