From 553ef1be94dffdedbc6e1fce9de91f77e5f12b95 Mon Sep 17 00:00:00 2001 From: Evan Cheng Date: Thu, 25 May 2006 20:16:55 +0000 Subject: [PATCH] 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 --- utils/TableGen/DAGISelEmitter.cpp | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) 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";