[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: lib/49745: tests/include/sys/t_bitops: improvement
At Sat, 14 Mar 2015 14:20:00 +0000 (UTC),
Christos Zoulas wrote:
> | >Description:
> | The current ilog2()'s implementation consists of 32bit variable
> | part, 64bit variable part and constant part. But 'ilog2_basic'
> | test checks only 64bit variable. 32bit variable and constant
> | were not checked.
> I would prefer if you put the constants in an array and loop instead
> of inlining it.
It can not be changed to loop because it tests
__builtin_constant_p() part of ilog2(). If all these inline tests
passed, ATF_TC_BODY(ilog2_const, tc) is compiled into
Tetsuya Isaki <isaki%pastel-flower.jp@localhost / isaki%NetBSD.org@localhost>
Main Index |
Thread Index |