[PATCH] D138924: [test-suite][tsvc] Use fabs in s318 and s3113
Sjoerd Meijer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 29 08:28:45 PST 2022
SjoerdMeijer added inline comments.
================
Comment at: MultiSource/Benchmarks/TSVC/tsc.inc:3778
end_t = clock(); clock_dif = end_t - start_t;
clock_dif_sec = (double) (clock_dif/1000000.0);
printf("S318\t %.2f \t\t", clock_dif_sec);;
----------------
rengolin wrote:
> UFF! This should really be `CLOCKS_PER_SEC`... But it's ok to make that a sweeping mechanical change on another patch.
Wowsers!!! Good catch.
This is fine though if we divide by the same random number everywhere. ;-)
But while I am at it, I could do the find and replace as a follow up.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138924/new/
https://reviews.llvm.org/D138924
More information about the llvm-commits
mailing list