[PATCH 3/3] Add new pow implementation

Szabolcs Nagy szabolcs.nagy@arm.com
Wed Sep 19 09:19:00 GMT 2018


On 17/09/18 23:19, Joseph Myers wrote:
> On Tue, 11 Sep 2018, Szabolcs Nagy wrote:
> 
>> +/* Returns 0 if not int, 1 if odd int, 2 if even int.  */
>> +static inline int
>> +checkint (uint64_t iy)
> 
> The comment needs to explain that this function is not called with an
> argument representing floating-point 0, Inf or NaN (since it wouldn't be
> correct for 0 at least).
> 
> OK with that change.
> 

Thanks for the reviews, i committed the patch and
separately fixed the comment in powf too in the
same way with the attached obvious patch.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-the-documentation-comment-of-checkint-in-powf.patch
Type: text/x-patch
Size: 1367 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20180919/57bce87a/attachment.bin>


More information about the Libc-alpha mailing list