[clang] [clang][bytecode][NFC] Use a ListSeparator to print backtraces (PR #177148)

via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 21 03:47:54 PST 2026


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Timm Baeder (tbaederr)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/177148.diff


1 Files Affected:

- (modified) clang/lib/AST/ByteCode/InterpFrame.cpp (+2-2) 


``````````diff
diff --git a/clang/lib/AST/ByteCode/InterpFrame.cpp b/clang/lib/AST/ByteCode/InterpFrame.cpp
index 3b883761ad001..5d645feb5579d 100644
--- a/clang/lib/AST/ByteCode/InterpFrame.cpp
+++ b/clang/lib/AST/ByteCode/InterpFrame.cpp
@@ -191,15 +191,15 @@ void InterpFrame::describe(llvm::raw_ostream &OS) const {
   Off += Func->hasRVO() ? primSize(PT_Ptr) : 0;
   Off += Func->hasThisPointer() ? primSize(PT_Ptr) : 0;
 
+  llvm::ListSeparator Comma;
   for (unsigned I = 0, N = F->getNumParams(); I < N; ++I) {
+    OS << Comma;
     QualType Ty = F->getParamDecl(I)->getType();
 
     PrimType PrimTy = S.Ctx.classify(Ty).value_or(PT_Ptr);
 
     TYPE_SWITCH(PrimTy, print(OS, stackRef<T>(Off), S.getASTContext(), Ty));
     Off += align(primSize(PrimTy));
-    if (I + 1 != N)
-      OS << ", ";
   }
   OS << ")";
 }

``````````

</details>


https://github.com/llvm/llvm-project/pull/177148


More information about the cfe-commits mailing list