[PATCH 3/7 v2] New generic log2f
Szabolcs Nagy
szabolcs.nagy@arm.com
Thu Sep 28 09:11:00 GMT 2017
On 27/09/17 19:03, Joseph Myers wrote:
> In the logf patch, you define LOGF_POLY_ORDER to 4, but actually only have
> 3 terms in the polynomial. In the log2f patch, you define
> LOG2F_POLY_ORDER to 4, and actually have and use 4 terms. Why should
> log2f need a larger polynomial than logf?
>
the order is meant to be the order of the polynomial
which is 4.
but in case of logf the first order coeff is fixed to
1.0 so we don't have to multiply with that.
i guess i should use
double poly[LOGF_POLY_ORDER - 1]; /* First order coefficient is 1. */
in the __logf_data declaration.
(these macros had more relevance when i had several
implementations of the functions with different table
size and poly order.)
More information about the Libc-alpha
mailing list