Re: [PATCH] Fix several AVX512 intrinsic mask arguments.

2018-07-06 Thread Jakub Jelinek
On Thu, Jul 05, 2018 at 11:57:26PM +0300, Grazvydas Ignotas wrote: > I think it would be more efficient if you took care of it. I won't > have time for at least a few days anyway. Here is what I have ATM, but will still need to work on the testsuite side for it and bootstrap/regtest it. In

Re: [PATCH] Fix several AVX512 intrinsic mask arguments.

2018-07-05 Thread Grazvydas Ignotas
On Thu, Jul 5, 2018 at 9:28 PM, Jakub Jelinek wrote: > On Thu, Jul 05, 2018 at 08:30:27PM +0300, Grazvydas Ignotas wrote: >> gcc/ChangeLog: >> >> 2018-07-05 Grazvydas Ignotas >> >> * config/i386/avx512bwintrin.h: (_mm512_mask_cmp_epi8_mask, >> _mm512_mask_cmp_epu8_mask): Fix mask

Re: [PATCH] Fix several AVX512 intrinsic mask arguments.

2018-07-05 Thread Jakub Jelinek
On Thu, Jul 05, 2018 at 08:30:27PM +0300, Grazvydas Ignotas wrote: > gcc/ChangeLog: > > 2018-07-05 Grazvydas Ignotas > > * config/i386/avx512bwintrin.h: (_mm512_mask_cmp_epi8_mask, > _mm512_mask_cmp_epu8_mask): Fix mask arguments. LGTM, but 1) I think it would be nice to add a

[PATCH] Fix several AVX512 intrinsic mask arguments.

2018-07-05 Thread Grazvydas Ignotas
gcc/ChangeLog: 2018-07-05 Grazvydas Ignotas * config/i386/avx512bwintrin.h: (_mm512_mask_cmp_epi8_mask, _mm512_mask_cmp_epu8_mask): Fix mask arguments. --- gcc/config/i386/avx512bwintrin.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git