[vlc-devel] [PATCH 1/2] vlc_common: builtin overflow functions are supported since gcc 6

Rémi Denis-Courmont remi at remlab.net
Thu Nov 23 17:21:22 CET 2017


Le torstaina 23. marraskuuta 2017, 17.16.59 EET KO Myung-Hun a écrit :
> ---
>  include/vlc_common.h | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/include/vlc_common.h b/include/vlc_common.h
> index 925c15fc2e..a149fc2696 100644
> --- a/include/vlc_common.h
> +++ b/include/vlc_common.h
> @@ -653,7 +653,7 @@ static inline uint64_t (bswap64)(uint64_t x)
>  /* Integer overflow */
>  static inline bool uadd_overflow(unsigned a, unsigned b, unsigned *res)
>  {
> -#if defined(__GNUC__) || defined(__clang__)
> +#if (defined(__GNUC__) && VLC_GCC_VERSION(5,0)) || defined(__clang__)

This is redundant. Same below.

>       return __builtin_uadd_overflow(a, b, res);
>  #else
>       *res = a + b;
> @@ -664,7 +664,7 @@ static inline bool uadd_overflow(unsigned a, unsigned b,
> unsigned *res) static inline bool uaddl_overflow(unsigned long a, unsigned
> long b, unsigned long *res)
>  {
> -#if defined(__GNUC__) || defined(__clang__)
> +#if (defined(__GNUC__) && VLC_GCC_VERSION(5,0)) || defined(__clang__)
>       return __builtin_uaddl_overflow(a, b, res);
>  #else
>       *res = a + b;
> @@ -675,7 +675,7 @@ static inline bool uaddl_overflow(unsigned long a,
> unsigned long b, static inline bool uaddll_overflow(unsigned long long a,
> unsigned long long b, unsigned long long *res)
>  {
> -#if defined(__GNUC__) || defined(__clang__)
> +#if (defined(__GNUC__) && VLC_GCC_VERSION(5,0)) || defined(__clang__)
>       return __builtin_uaddll_overflow(a, b, res);
>  #else
>       *res = a + b;
> @@ -710,7 +710,7 @@ static inline bool add_overflow(unsigned long long a,
> unsigned long long b,
> 
>  static inline bool umul_overflow(unsigned a, unsigned b, unsigned *res)
>  {
> -#if defined(__GNUC__) || defined(__clang__)
> +#if (defined(__GNUC__) && VLC_GCC_VERSION(5,0)) || defined(__clang__)
>       return __builtin_umul_overflow(a, b, res);
>  #else
>       *res = a * b;
> @@ -721,7 +721,7 @@ static inline bool umul_overflow(unsigned a, unsigned b,
> unsigned *res) static inline bool umull_overflow(unsigned long a, unsigned
> long b, unsigned long *res)
>  {
> -#if defined(__GNUC__) || defined(__clang__)
> +#if (defined(__GNUC__) && VLC_GCC_VERSION(5,0)) || defined(__clang__)
>       return __builtin_umull_overflow(a, b, res);
>  #else
>       *res = a * b;
> @@ -732,7 +732,7 @@ static inline bool umull_overflow(unsigned long a,
> unsigned long b, static inline bool umulll_overflow(unsigned long long a,
> unsigned long long b, unsigned long long *res)
>  {
> -#if defined(__GNUC__) || defined(__clang__)
> +#if (defined(__GNUC__) && VLC_GCC_VERSION(5,0)) || defined(__clang__)
>       return __builtin_umulll_overflow(a, b, res);
>  #else
>       *res = a * b;


-- 
雷米‧德尼-库尔蒙
https://www.remlab.net/



More information about the vlc-devel mailing list