Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace RAPIDJSON_CLZLL with internal clzll #1660

Merged
merged 1 commit into from
Mar 11, 2020
Merged

Commits on Mar 10, 2020

  1. 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: Iac4b355dc5e5b4ed9b3f35a640b6b5537e76f22c
    Signed-off-by: Jun He <jun.he@arm.com>
    JunHe77 committed Mar 10, 2020
    Configuration menu
    Copy the full SHA
    cb7052b View commit details
    Browse the repository at this point in the history