Re: [hackers] [libgrapheme] Mark likely branches || Laslo Hunhold

[NRK]

> +#ifdef __has_builtin
> +     #if __has_builtin(__builtin_expect)
> +             #define likely(expr) __builtin_expect(!!(expr), 1)
> +             #define unlikely(expr) __builtin_expect(!!(expr), 0)
> +     #else
> +             #define likely(expr) (expr)
> +             #define unlikely(expr) (expr)
> +     #endif
> +#else
> +     #define likely(expr) (expr)
> +     #define unlikely(expr) (expr)
> +#endif

Just curious, why not use:

#if defined(__has_builtin) && __has_builtin(__builtin_expect)
        #define likely(expr) __builtin_expect(!!(expr), 1)
        #define unlikely(expr) __builtin_expect(!!(expr), 0)
#else
        #define likely(expr) (expr)
        #define unlikely(expr) (expr)
#endif


- NRK