Skip to content

Commit 96eff99

Browse files
[llvm-reduce] Avoid repeated hash lookups (NFC) (#109747)
1 parent cd53c84 commit 96eff99

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

llvm/tools/llvm-reduce/deltas/ReduceDistinctMetadata.cpp

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,8 @@ reduceNodes(MDNode *Root,
4141
if (MDNode *Operand =
4242
dyn_cast<MDNode>(CurrentNode->getOperand(I).get())) {
4343
// Check whether node has been visited
44-
if (!VisitedNodes.contains(Operand)) {
44+
if (VisitedNodes.insert(Operand))
4545
NodesToTraverse.push(Operand);
46-
VisitedNodes.insert(Operand);
47-
}
4846
// Delete the node only if it is distinct
4947
if (Operand->isDistinct()) {
5048
// Add to removal list
@@ -74,10 +72,8 @@ static void cleanUpTemporaries(NamedMDNode &NamedNode, MDTuple *TemporaryTuple,
7472
// If the node hasn't been traversed yet, add it to the queue of nodes to
7573
// traverse.
7674
if (MDTuple *TupleI = dyn_cast<MDTuple>((*I))) {
77-
if (!VisitedNodes.contains(TupleI)) {
75+
if (VisitedNodes.insert(TupleI))
7876
NodesToTraverse.push(TupleI);
79-
VisitedNodes.insert(TupleI);
80-
}
8177
}
8278
}
8379

@@ -113,12 +109,10 @@ static void cleanUpTemporaries(NamedMDNode &NamedNode, MDTuple *TemporaryTuple,
113109
// Push the remaining nodes into the queue
114110
for (unsigned int I = 0; I < CurrentTuple->getNumOperands(); ++I) {
115111
MDTuple *Operand = dyn_cast<MDTuple>(CurrentTuple->getOperand(I).get());
116-
if (Operand && !VisitedNodes.contains(Operand)) {
117-
NodesToTraverse.push(Operand);
112+
if (Operand && VisitedNodes.insert(Operand))
118113
// If the node hasn't been traversed yet, add it to the queue of nodes
119114
// to traverse.
120-
VisitedNodes.insert(Operand);
121-
}
115+
NodesToTraverse.push(Operand);
122116
}
123117
}
124118
}

0 commit comments

Comments
 (0)