Hi! On Mon, Oct 22, 2018 at 01:33:18PM -0500, Bill Schmidt wrote: > We should never use __vector long in the intrinsics compatibility headers, > since this has different meanings on 32- and 64-bit targets. This patch > corrects a number of places where this previously slipped through.
> --- gcc/config/rs6000/emmintrin.h (revision 265389) > +++ gcc/config/rs6000/emmintrin.h (working copy) > @@ -888,7 +888,8 @@ _mm_cvtpd_epi32 (__m128d __A) > > #ifdef _ARCH_PWR8 > temp = vec_mergeo (temp, temp); > - result = (__v4si)vec_vpkudum ((__vector long)temp, (__vector long)vzero); > + result = (__v4si)vec_vpkudum ((__vector long long)temp, > + (__vector long long)vzero); The old code had this as well, but there should be a space after a cast. Okay for trunk with or without that fixed up. Thanks! Segher
