Skip to content

Commit 82e93ae

Browse files
committed
minor formatting change
Signed-off-by: Magnus Lundmark <[email protected]>
1 parent c2ccc1b commit 82e93ae

File tree

2 files changed

+4
-18
lines changed

2 files changed

+4
-18
lines changed

lib/kernel_tests.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,8 @@ std::vector<volk_test_case_t> init_test_list(volk_test_params_t test_params)
232232
QA(VOLK_INIT_TEST(volk_32f_sqrt_32f, test_params_inacc))
233233

234234
volk_test_params_t test_params_invsqrt(test_params.make_tol(1e-6));
235-
test_params_invsqrt.add_float_edge_cases({ -1.f, 0.f, inf, 0.01f, 100.0f });
235+
test_params_invsqrt.add_float_edge_cases(
236+
{ -1.f, 1.f, 0.f, inf, 1e-2f, 1e2f, 1e-10, 1e10 });
236237
QA(VOLK_INIT_TEST(volk_32f_invsqrt_32f, test_params_invsqrt))
237238
QA(VOLK_INIT_TEST(volk_32f_s32f_stddev_32f, test_params_inacc))
238239
QA(VOLK_INIT_TEST(volk_32f_stddev_and_mean_32f_x2, test_params.make_absolute(1e-5)))

lib/qa_utils.cc

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1451,20 +1451,10 @@ bool run_volk_tests(volk_func_desc_t desc,
14511451

14521452
// Helper for adaptive decimal places based on magnitude
14531453
auto format_time = [](double ms) -> std::string {
1454-
if (ms >= 10000.0)
1455-
return fmt::format("{:.1f} ms", ms);
1456-
if (ms >= 1000.0)
1457-
return fmt::format("{:.2f} ms", ms);
1458-
if (ms >= 100.0)
1459-
return fmt::format("{:.3f} ms", ms);
1460-
return fmt::format("{:.4f} ms", ms);
1454+
return fmt::format("{:.2f} ms", ms);
14611455
};
14621456

14631457
auto format_throughput = [](double mbps) -> std::string {
1464-
if (mbps >= 100000.0)
1465-
return fmt::format("{:.0f} MB/s", mbps);
1466-
if (mbps >= 10000.0)
1467-
return fmt::format("{:.1f} MB/s", mbps);
14681458
return fmt::format("{:.1f} MB/s", mbps);
14691459
};
14701460

@@ -1504,12 +1494,7 @@ bool run_volk_tests(volk_func_desc_t desc,
15041494
speedup_str = "-";
15051495
} else {
15061496
double speedup = generic_time / profile_times[i];
1507-
if (speedup >= 100.0)
1508-
speedup_str = fmt::format("{:.1f}x", speedup);
1509-
else if (speedup >= 10.0)
1510-
speedup_str = fmt::format("{:.2f}x", speedup);
1511-
else
1512-
speedup_str = fmt::format("{:.2f}x", speedup);
1497+
speedup_str = fmt::format("{:.2f}x", speedup);
15131498
}
15141499
std::string err_str =
15151500
(arch_list[i] == "generic") ? "-" : fmt::format("{:.1e}", arch_max_err[i]);

0 commit comments

Comments
 (0)