Hi!

On Wed, Oct 24, 2018 at 02:30:10PM -0500, Bill Schmidt wrote:
> Due to some unfortunate history, not all compilers currently have correct 
> result
> types produced for comparison operators on vector types.  For compatibility
> purposes, this patch replaces those with vec_cmp* built-ins.  It also cleans 
> up
> some silly type usages and formatting in the affected functions.

Those compilers really need to be fixed, this isn't a tiny detail they
get wrong.

But, this patch makes the code more readable, so okay.  Thanks!


Segher


> 2018-10-24  Bill Schmidt  <wschm...@linux.ibm.com>
>           Jinsong Ji <j...@us.ibm.com>
> 
>       * gcc/config/rs6000/emmintrin.h (_mm_sll_epi16): Replace
>       comparison operators with vec_cmp* for compatibility due to
>       unfortunate history; clean up formatting and use types more
>       appropriately.
>       (_mm_sll_epi32): Likewise.
>       (_mm_sll_epi64): Likewise.
>       (_mm_srl_epi16): Likewise.
>       (_mm_srl_epi32): Likewise.
>       (_mm_srl_epi64): Likewise.

Reply via email to