[PATCH] Two small i?86 *intrin* warning fixes

Uros Bizjak ubizjak@gmail.com
Thu Jan 30 18:00:00 GMT 2014


On Thu, Jan 30, 2014 at 6:52 PM, Jakub Jelinek <jakub@redhat.com> wrote:

> While looking at some other PR, I've stripped line notes and got
> pr59947.ii.bak:26330:74: error: ISO C++ forbids declaration of '_mm512_mask_cvtusepi64_storeu_epi32' with no type [-fpermissive]
>  _mm512_mask_cvtusepi64_storeu_epi32 (void* __P, __mmask8 __M, __m512i __A)
>                                                                           ^
> pr59947.ii.bak: In function 'float _cvtsh_ss(short unsigned int)':
> pr59947.ii.bak:30674:65: warning: narrowing conversion of '__S' from 'short unsigned int' to 'short int' inside { } [-Wnarrowing]
>    __v8hi __H = __extension__ (__v8hi){ __S, 0, 0, 0, 0, 0, 0, 0 };
>                                                                  ^
> warnings that would normally only show up with -Wsystem-headers.
> Especially the second one looks like one worth fixing.
>
> Ok for trunk?
>
> 2014-01-30  Jakub Jelinek  <jakub@redhat.com>
>
>         * config/i386/f16cintrin.h (_cvtsh_ss): Avoid -Wnarrowing
>         warning.
>         * config/i386/avx512fintrin.h (_mm512_mask_cvtusepi64_storeu_epi32):
>         Add missing return type - void.

OK.

Should _cvtsh_ss fix be backported to other release branches?

Thanks,
Uros.



More information about the Gcc-patches mailing list