Tencent / rapidjson / 2267

100%

master: 100%

Pull #1660

travis-ci

Replace RAPIDJSON_CLZLL with internal clzll

RAPIDJSON_CLZLL is defined as macro of __builtin_clzll when
using gcc to compile. This introduces two issues:
1. in gcc __builtin_clzll returns int, not uint32_t.
2. __builtin_clzll return is undefined when input x is 0
See: https://gcc.gnu.org/onlinedocs/gcc/Other-Builtins.html

This patch removes RAPIDJSON_CLZLL, merges __builtin_clzll to
internal clzll with input check and return value explicit cast.

Change-Id: Iac4b355dc
Signed-off-by: Jun He <jun.he@arm.com>

2 of 2 new or added lines in 2 files covered. (100.0%)

4982 of 4986 relevant lines covered (99.92%)

1773221.89 hits per line