Skip to content

Commit

Permalink
chore(benchmark): fix invalid result message
Browse files Browse the repository at this point in the history
  • Loading branch information
ashjeong committed Oct 18, 2024
1 parent caab54e commit 779f070
Show file tree
Hide file tree
Showing 7 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion benchmark/fft/fft_benchmark.cc
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ template <typename PolyOrEvals>
void CheckResult(bool check_result, const PolyOrEvals& tachyon_result,
const PolyOrEvals& vendor_result) {
if (check_result) {
CHECK_EQ(tachyon_result, vendor_result) << "Results not matched";
CHECK_EQ(tachyon_result, vendor_result) << "Results do not match";
}
}

Expand Down
2 changes: 1 addition & 1 deletion benchmark/fft/fft_benchmark_gpu.cc
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ void Run(const FFTConfig& config) {
domain.get(), input, kShouldRecord, gpu_result);
}
if (config.check_results()) {
CHECK_EQ(cpu_result, gpu_result) << "Results not matched";
CHECK_EQ(cpu_result, gpu_result) << "Results do not match";
}
}

Expand Down
2 changes: 1 addition & 1 deletion benchmark/fft_batch/fft_batch_benchmark.cc
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ void CheckResults(bool check_results,
const math::RowMajorMatrix<F>& tachyon_result,
const math::RowMajorMatrix<F>& vendor_result) {
if (check_results) {
CHECK_EQ(tachyon_result, vendor_result) << "Results not matched";
CHECK_EQ(tachyon_result, vendor_result) << "Results do not match";
}
}

Expand Down
2 changes: 1 addition & 1 deletion benchmark/fri/fri_benchmark.cc
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ template <typename Result>
void CheckResult(bool check_results, const Result& tachyon_result,
const Result& vendor_result) {
if (check_results) {
CHECK_EQ(tachyon_result, vendor_result) << "Results not matched";
CHECK_EQ(tachyon_result, vendor_result) << "Results do not match";
}
}

Expand Down
2 changes: 1 addition & 1 deletion benchmark/msm/msm_benchmark.cc
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ int RealMain(int argc, char** argv) {
}

if (config.check_results()) {
CHECK(results == results_vendor) << "Result not matched";
CHECK(results == results_vendor) << "Results do not match";
}
}

Expand Down
2 changes: 1 addition & 1 deletion benchmark/msm/msm_benchmark_gpu.cc
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ int RealMain(int argc, char** argv) {
tachyon_bn254_g1_destroy_msm_gpu(msm_gpu);

if (config.check_results()) {
CHECK(results_cpu == results_gpu) << "Result not matched";
CHECK(results_cpu == results_gpu) << "Results do not match";
}

reporter.Show();
Expand Down
2 changes: 1 addition & 1 deletion benchmark/poseidon/poseidon_benchmark.cc
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ int RealMain(int argc, char** argv) {
runner.RunExternal(Vendor::Arkworks(), run_poseidon_arkworks);

if (config.check_results()) {
CHECK_EQ(result, result_arkworks) << "Result not matched";
CHECK_EQ(result, result_arkworks) << "Results do not match";
}

reporter.AddAverageAsLastColumn();
Expand Down

0 comments on commit 779f070

Please sign in to comment.