Skip to content

Commit 9e4033b

Browse files
[TableGen] Use ListSeparator (NFC)
1 parent 9524632 commit 9e4033b

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

llvm/utils/TableGen/FastISelEmitter.cpp

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,9 @@ struct OperandsSignature {
290290
}
291291

292292
void PrintParameters(raw_ostream &OS) const {
293+
ListSeparator LS;
293294
for (unsigned i = 0, e = Operands.size(); i != e; ++i) {
295+
OS << LS;
294296
if (Operands[i].isReg()) {
295297
OS << "unsigned Op" << i << ", bool Op" << i << "IsKill";
296298
} else if (Operands[i].isImm()) {
@@ -300,39 +302,35 @@ struct OperandsSignature {
300302
} else {
301303
llvm_unreachable("Unknown operand kind!");
302304
}
303-
if (i + 1 != e)
304-
OS << ", ";
305305
}
306306
}
307307

308308
void PrintArguments(raw_ostream &OS,
309309
const std::vector<std::string> &PR) const {
310310
assert(PR.size() == Operands.size());
311-
bool PrintedArg = false;
311+
ListSeparator LS;
312312
for (unsigned i = 0, e = Operands.size(); i != e; ++i) {
313313
if (PR[i] != "")
314314
// Implicit physical register operand.
315315
continue;
316316

317-
if (PrintedArg)
318-
OS << ", ";
317+
OS << LS;
319318
if (Operands[i].isReg()) {
320319
OS << "Op" << i << ", Op" << i << "IsKill";
321-
PrintedArg = true;
322320
} else if (Operands[i].isImm()) {
323321
OS << "imm" << i;
324-
PrintedArg = true;
325322
} else if (Operands[i].isFP()) {
326323
OS << "f" << i;
327-
PrintedArg = true;
328324
} else {
329325
llvm_unreachable("Unknown operand kind!");
330326
}
331327
}
332328
}
333329

334330
void PrintArguments(raw_ostream &OS) const {
331+
ListSeparator LS;
335332
for (unsigned i = 0, e = Operands.size(); i != e; ++i) {
333+
OS << LS;
336334
if (Operands[i].isReg()) {
337335
OS << "Op" << i << ", Op" << i << "IsKill";
338336
} else if (Operands[i].isImm()) {
@@ -342,8 +340,6 @@ struct OperandsSignature {
342340
} else {
343341
llvm_unreachable("Unknown operand kind!");
344342
}
345-
if (i + 1 != e)
346-
OS << ", ";
347343
}
348344
}
349345

0 commit comments

Comments
 (0)