[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