From 3bdc4bdba61f44e06f1d35e2ee23403e585eb324 Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Fri, 20 Mar 2009 20:42:23 +0000 Subject: Simplify this code; use a while instead of an if and a do-while. llvm-svn: 67400 --- llvm/lib/CodeGen/SelectionDAG/ScheduleDAGSDNodes.cpp | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) (limited to 'llvm/lib/CodeGen') diff --git a/llvm/lib/CodeGen/SelectionDAG/ScheduleDAGSDNodes.cpp b/llvm/lib/CodeGen/SelectionDAG/ScheduleDAGSDNodes.cpp index ea9f47bf318..967a8cef8e1 100644 --- a/llvm/lib/CodeGen/SelectionDAG/ScheduleDAGSDNodes.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/ScheduleDAGSDNodes.cpp @@ -105,18 +105,15 @@ void ScheduleDAGSDNodes::BuildSchedUnits() { // See if anything is flagged to this node, if so, add them to flagged // nodes. Nodes can have at most one flag input and one flag output. Flags - // are required the be the last operand and result of a node. + // are required to be the last operand and result of a node. // Scan up to find flagged preds. SDNode *N = NI; - if (N->getNumOperands() && - N->getOperand(N->getNumOperands()-1).getValueType() == MVT::Flag) { - do { - N = N->getOperand(N->getNumOperands()-1).getNode(); - assert(N->getNodeId() == -1 && "Node already inserted!"); - N->setNodeId(NodeSUnit->NodeNum); - } while (N->getNumOperands() && - N->getOperand(N->getNumOperands()-1).getValueType()== MVT::Flag); + while (N->getNumOperands() && + N->getOperand(N->getNumOperands()-1).getValueType() == MVT::Flag) { + N = N->getOperand(N->getNumOperands()-1).getNode(); + assert(N->getNodeId() == -1 && "Node already inserted!"); + N->setNodeId(NodeSUnit->NodeNum); } // Scan down to find any flagged succs. -- cgit v1.2.3