[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