[llvm] [PassManager] Use ListSeparator. NFC (PR #173462)

Zakk Chen via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 24 17:55:15 PST 2025


https://github.com/zakk0610 updated https://github.com/llvm/llvm-project/pull/173462

>From 67a84a8b0f83e7d177d96b8cfe9571301b40f425 Mon Sep 17 00:00:00 2001
From: Zakk Chen <zakkc at google.com>
Date: Wed, 24 Dec 2025 07:06:34 +0000
Subject: [PATCH] [PassManager] Use ListSeparator. NFC

Change-Id: I40fa748d2c188171caccff7cb418c6be74c22fb8
---
 llvm/include/llvm/IR/PassManager.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/llvm/include/llvm/IR/PassManager.h b/llvm/include/llvm/IR/PassManager.h
index 4354551a2405b..34a55a0ae4eb7 100644
--- a/llvm/include/llvm/IR/PassManager.h
+++ b/llvm/include/llvm/IR/PassManager.h
@@ -39,6 +39,7 @@
 
 #include "llvm/ADT/DenseMap.h"
 #include "llvm/ADT/STLExtras.h"
+#include "llvm/ADT/StringExtras.h"
 #include "llvm/ADT/StringRef.h"
 #include "llvm/ADT/TinyPtrVector.h"
 #include "llvm/IR/Analysis.h"
@@ -177,11 +178,10 @@ class PassManager : public PassInfoMixin<
 
   void printPipeline(raw_ostream &OS,
                      function_ref<StringRef(StringRef)> MapClassName2PassName) {
-    for (unsigned Idx = 0, Size = Passes.size(); Idx != Size; ++Idx) {
-      auto *P = Passes[Idx].get();
+    ListSeparator LS(",");
+    for (auto &P : Passes) {
+      OS << LS;
       P->printPipeline(OS, MapClassName2PassName);
-      if (Idx + 1 < Size)
-        OS << ',';
     }
   }
 



More information about the llvm-commits mailing list