From bd2b6214862ccc33d3bb63005faa15c16ee51c49 Mon Sep 17 00:00:00 2001 From: Jim Laskey Date: Sun, 18 Dec 2005 04:40:52 +0000 Subject: [PATCH] Fix a bug Sabre was having where the DAG root was a group. The group dominator needed to be added to the ordering list, not the first member of the group. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@24816 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/SelectionDAG/ScheduleDAG.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/CodeGen/SelectionDAG/ScheduleDAG.cpp b/lib/CodeGen/SelectionDAG/ScheduleDAG.cpp index ef90fc048c5..03a0dd1200c 100644 --- a/lib/CodeGen/SelectionDAG/ScheduleDAG.cpp +++ b/lib/CodeGen/SelectionDAG/ScheduleDAG.cpp @@ -682,7 +682,12 @@ void SimpleSched::IncludeNode(NodeInfo *NI) { /// Note that the ordering in the Nodes vector is reversed. void SimpleSched::VisitAll() { // Add first element to list - Ordering.push_back(getNI(DAG.getRoot().Val)); + NodeInfo *NI = getNI(DAG.getRoot().Val); + if (NI->isInGroup()) { + Ordering.push_back(NI->Group->getDominator()); + } else { + Ordering.push_back(NI); + } // Iterate through all nodes that have been added for (unsigned i = 0; i < Ordering.size(); i++) { // note: size() varies