[v4.1][PATCH] Framework for performance benchmarking of functions

Siddhesh Poyarekar siddhesh.poyarekar@gmail.com
Thu Mar 14 17:19:00 GMT 2013


On 14 March 2013 22:38, Richard Henderson <rth@twiddle.net> wrote:
> You should have tested at least i386, because you still didn't have the right
> types in the printf for 32-bit.

Sorry about that.  I had assumed that Tulio's patch had fixed that on
powerpc32, but I should have verified it.

>> ./bench-skeleton.c:70:11: warning: format ‘%lld’ expects argument of type ‘long long int’, but argument 2 has type ‘unsigned int’ [-Wformat]
>
> Here's an incremental patch that fixes that and, IMO, makes the results more
> readable.

Thanks!  Would it be OK to commit this and your patch or would you
like me to respin the patch for review?

Siddhesh
-- 
http://siddhesh.in



More information about the Libc-alpha mailing list