Hi!

On Mon, 19 Mar 2018 20:35:56 +0100, Jakub Jelinek <ja...@redhat.com> wrote:
> --- libgcc/config/i386/cpuinfo.c.jj   2018-03-15 09:10:20.870075051 +0100
> +++ libgcc/config/i386/cpuinfo.c      2018-03-19 16:13:25.059481079 +0100
> @@ -231,78 +238,81 @@ get_available_features (unsigned int ecx
>    unsigned int ext_level;
>  
>    unsigned int features = 0;
> +  unsigned int features2 = 0;
>  
> +#define set_feature(f) \
> +  if (f < 32) features |= (1U << f); else features2 |= (1U << (f - 32))
>    if (edx & bit_CMOV)
> -    features |= (1 << FEATURE_CMOV);
> +    set_feature (FEATURE_CMOV);

    [...]/libgcc/config/i386/cpuinfo.c: In function 'get_available_features':
    [...]/libgcc/config/i386/cpuinfo.c:278:60: warning: left shift count is 
negative [-Wshift-count-negative]
       if (f < 32) features |= (1U << f); else features2 |= (1U << (f - 32))
                                                                ^~
    [...]/libgcc/config/i386/cpuinfo.c:281:5: note: in expansion of macro 
'set_feature'
         set_feature (FEATURE_CMOV);
         ^~~~~~~~~~~
    [...]/libgcc/config/i386/cpuinfo.c:280:6: warning: suggest explicit braces 
to avoid ambiguous 'else' [-Wdangling-else]
       if (edx & bit_CMOV)
          ^
    [Many more.]


Grüße
 Thomas

Reply via email to