[v3] Add math-inline benchmark

Wilco Dijkstra wdijkstr@arm.com
Wed Sep 16 15:27:00 GMT 2015


> Siddhesh Poyarekar wrote:
> Sorry, I assumed someone else (Carlos?) was reviewing this and then
> forgot about it.  I've put review comments inline.  The major concern I
> have is that you're duplicating contents of math_private.h to compare
> it against the gcc builtins, which is fine as a first hack if you want
> to avoid changing actual code for the tests, but may not be a good idea
> in the long term.  That is, there is a risk that the inlines may be
> modified but this test doesn't realize that.
> 
> I don't mind this going in as a first revision (with other changes
> recommended below) provided you're willing to remedy this situation in
> the next iteration so that there is at least a failure in running the
> benchtest if the definitions of the inlines change in any way.

I previously posted a patch to enable inlines in math.h which replace the
GLIBC internal inlines. I also posted a patch to rename all uses of
__isinf_ns to isinf so they also use the new inline. I am going to post a 
patch remove all the math_private inlines as they are never used after
these changes.

So these functions are not duplicated, and it is necessary to add a copy
to benchmark the old implementation. Ie. the only duplication is 
EXTRACT_WORDS64, which seems fairly minor.

I've updated my patch with your other comments, see attached.

Wilco

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-Add-bench-math-inlines.txt
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20150916/0f70bb38/attachment.txt>


More information about the Libc-alpha mailing list