Re: lib/49745: tests/include/sys/t_bitops: improvement

[Tetsuya Isaki <isaki%pastel-flower.jp@localhost>]

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

  ATF_TC_BODY(ilog2_const, tc)
  {
  }

---
Tetsuya Isaki <isaki%pastel-flower.jp@localhost / isaki%NetBSD.org@localhost>