[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