[llvm] [NFC] simplify LowerAllowCheckPass::printPipeline (PR #149374)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 17 11:24:12 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
Author: Florian Mayer (fmayer)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/149374.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp (+6-13)
``````````diff
diff --git a/llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp b/llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp
index e7a6fa48e9004..55f3239ada100 100644
--- a/llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp
+++ b/llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp
@@ -10,6 +10,7 @@
#include "llvm/ADT/SmallVector.h"
#include "llvm/ADT/Statistic.h"
+#include "llvm/ADT/StringExtras.h"
#include "llvm/ADT/StringRef.h"
#include "llvm/Analysis/OptimizationRemarkEmitter.h"
#include "llvm/Analysis/ProfileSummaryInfo.h"
@@ -184,22 +185,14 @@ void LowerAllowCheckPass::printPipeline(
// correctness.
// TODO: print shorter output by combining adjacent runs, etc.
int i = 0;
- bool printed = false;
+ ListSeparator LS(";");
for (unsigned int cutoff : Opts.cutoffs) {
- if (cutoff > 0) {
- if (printed)
- OS << ";";
- OS << "cutoffs[" << i << "]=" << cutoff;
- printed = true;
- }
-
+ if (cutoff > 0)
+ OS << LS << "cutoffs[" << i << "]=" << cutoff;
i++;
}
- if (Opts.runtime_check) {
- if (printed)
- OS << ";";
- OS << "runtime_check=" << Opts.runtime_check;
- }
+ if (Opts.runtime_check)
+ OS << LS << "runtime_check=" << Opts.runtime_check;
OS << '>';
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/149374
More information about the llvm-commits
mailing list