[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