[llvm] [LLVM][Support] Minor code cleanup in `InterleavedRange` (PR #175237)

Rahul Joshi via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 9 12:18:23 PST 2026


https://github.com/jurahul created https://github.com/llvm/llvm-project/pull/175237

Remove redundant empty checks for prefix/suffix strings and flush() for raw_string_ostream.

>From 1e3be96dbbafe21e04cf11a27e45087b57d734ca Mon Sep 17 00:00:00 2001
From: Rahul Joshi <rjoshi at nvidia.com>
Date: Fri, 9 Jan 2026 12:16:26 -0800
Subject: [PATCH] [LLVM][Support] Minor code cleanup in `InterleavedRange`

Remove redundant empty checks for prefix/suffix strings and flush()
for raw_string_ostream.
---
 llvm/include/llvm/Support/InterleavedRange.h | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/llvm/include/llvm/Support/InterleavedRange.h b/llvm/include/llvm/Support/InterleavedRange.h
index ce7cac98bcdd4..2c7b87036f4d1 100644
--- a/llvm/include/llvm/Support/InterleavedRange.h
+++ b/llvm/include/llvm/Support/InterleavedRange.h
@@ -44,11 +44,9 @@ template <typename Range> class InterleavedRange {
 
   template <typename OStream>
   friend OStream &operator<<(OStream &OS, const InterleavedRange &Interleaved) {
-    if (!Interleaved.Prefix.empty())
-      OS << Interleaved.Prefix;
+    OS << Interleaved.Prefix;
     llvm::interleave(Interleaved.TheRange, OS, Interleaved.Separator);
-    if (!Interleaved.Suffix.empty())
-      OS << Interleaved.Suffix;
+    OS << Interleaved.Suffix;
     return OS;
   }
 
@@ -56,7 +54,6 @@ template <typename Range> class InterleavedRange {
     std::string Result;
     raw_string_ostream Stream(Result);
     Stream << *this;
-    Stream.flush();
     return Result;
   }
 



More information about the llvm-commits mailing list