[clang] [clang-tools-extra] [llvm] [llvm-exegesis] Replace --num-repetitions with --min-instructions (PR #77153)

via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 1 00:33:42 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 90e68086d8fdbfb32dfc7e7e3498f44365274ce8 6a81778059d72a4e4d236d0d6777a621a5bc965b -- llvm/tools/llvm-exegesis/lib/BenchmarkResult.cpp llvm/tools/llvm-exegesis/lib/BenchmarkResult.h llvm/tools/llvm-exegesis/lib/BenchmarkRunner.cpp llvm/tools/llvm-exegesis/lib/BenchmarkRunner.h llvm/tools/llvm-exegesis/lib/ResultAggregator.cpp llvm/tools/llvm-exegesis/llvm-exegesis.cpp llvm/unittests/tools/llvm-exegesis/Mips/BenchmarkResultTest.cpp llvm/unittests/tools/llvm-exegesis/ResultAggregatorTest.cpp llvm/unittests/tools/llvm-exegesis/X86/BenchmarkResultTest.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/tools/llvm-exegesis/lib/ResultAggregator.cpp b/llvm/tools/llvm-exegesis/lib/ResultAggregator.cpp
index 7e873bc9d4..f3470344a8 100644
--- a/llvm/tools/llvm-exegesis/lib/ResultAggregator.cpp
+++ b/llvm/tools/llvm-exegesis/lib/ResultAggregator.cpp
@@ -47,7 +47,8 @@ void MiddleHalfResultAggregator::AggregateMeasurement(
     const Benchmark &Result) const {
   Measurement.RawValue = NewMeasurement.RawValue - Measurement.RawValue;
   Measurement.PerInstructionValue = Measurement.RawValue;
-  Measurement.PerInstructionValue /= Result.MinInstructions;;
+  Measurement.PerInstructionValue /= Result.MinInstructions;
+  ;
   Measurement.PerSnippetValue = Measurement.RawValue;
   Measurement.PerSnippetValue /=
       std::ceil(Result.MinInstructions /

``````````

</details>


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


More information about the cfe-commits mailing list