[llvm] [NFC][LLVM] Adopt ListSeparator in more places (PR #172909)
Jason Eckhardt via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 19 14:54:49 PST 2025
================
@@ -1219,15 +1220,10 @@ Error DWARFDebugLine::LineTable::parse(
}
if (Verbose && !Operands.empty()) {
*OS << " (operands: ";
- bool First = true;
- for (uint64_t Value : Operands) {
- if (!First)
- *OS << ", ";
- First = false;
- *OS << format("0x%16.16" PRIx64, Value);
- }
- if (Verbose)
- *OS << ')';
+ ListSeparator LS;
+ for (uint64_t Value : Operands)
+ *OS << LS << format("0x%16.16" PRIx64, Value);
+ *OS << ')';
----------------
nvjle wrote:
As elsewhere, `formatv` to avoid explicit looping instead?
```
*OS << formatv("{:x16}", make_range(Operands.begin(), Operands.end()));
```
https://github.com/llvm/llvm-project/pull/172909
More information about the llvm-commits
mailing list