Make addPred and removePred return void, since the return value is not

currently used by anything.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61066 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Dan Gohman 2008-12-16 01:00:55 +00:00
parent 3ee7449f39
commit ffa391272b
3 changed files with 14 additions and 17 deletions

View File

@ -299,12 +299,12 @@ namespace llvm {
/// addPred - This adds the specified edge as a pred of the current node if
/// not already. It also adds the current node as a successor of the
/// specified node. This returns true if this is a new pred.
bool addPred(const SDep &D) {
/// specified node.
void addPred(const SDep &D) {
// If this node already has this depenence, don't add a redundant one.
for (unsigned i = 0, e = (unsigned)Preds.size(); i != e; ++i)
if (Preds[i] == D)
return false;
return;
// Add a pred to this SUnit.
Preds.push_back(D);
// Now add a corresponding succ to N.
@ -321,14 +321,12 @@ namespace llvm {
++NumPredsLeft;
if (!isScheduled)
++N->NumSuccsLeft;
return true;
}
/// removePred - This removes the specified edge as a pred of the current
/// node if it exists. It also removes the current node as a successor of
/// the specified node. This returns true if the edge existed and was
/// removed.
bool removePred(const SDep &D) {
/// the specified node.
void removePred(const SDep &D) {
// Find the matching predecessor.
for (SmallVector<SDep, 4>::iterator I = Preds.begin(), E = Preds.end();
I != E; ++I)
@ -356,9 +354,8 @@ namespace llvm {
--NumPredsLeft;
if (!isScheduled)
--N->NumSuccsLeft;
return true;
return;
}
return false;
}
bool isPred(SUnit *N) {

View File

@ -79,14 +79,14 @@ public:
/// AddPred - adds a predecessor edge to SUnit SU.
/// This returns true if this is a new predecessor.
bool AddPred(SUnit *SU, const SDep &D) {
return SU->addPred(D);
void AddPred(SUnit *SU, const SDep &D) {
SU->addPred(D);
}
/// RemovePred - removes a predecessor edge from SUnit SU.
/// This returns true if an edge was removed.
bool RemovePred(SUnit *SU, const SDep &D) {
return SU->removePred(D);
void RemovePred(SUnit *SU, const SDep &D) {
SU->removePred(D);
}
private:

View File

@ -101,17 +101,17 @@ public:
/// AddPred - adds a predecessor edge to SUnit SU.
/// This returns true if this is a new predecessor.
/// Updates the topological ordering if required.
bool AddPred(SUnit *SU, const SDep &D) {
void AddPred(SUnit *SU, const SDep &D) {
Topo.AddPred(SU, D.getSUnit());
return SU->addPred(D);
SU->addPred(D);
}
/// RemovePred - removes a predecessor edge from SUnit SU.
/// This returns true if an edge was removed.
/// Updates the topological ordering if required.
bool RemovePred(SUnit *SU, const SDep &D) {
void RemovePred(SUnit *SU, const SDep &D) {
Topo.RemovePred(SU, D.getSUnit());
return SU->removePred(D);
SU->removePred(D);
}
private: