Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-18 Thread Kirill Yukhin
Hello Uroš, On 17 Feb 13:41, Uros Bizjak wrote: On Mon, Feb 17, 2014 at 1:26 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Please don't change srcp pattern, it should be defined similar to vrcpss (aka sse_vmrcpv4sf). You need to switch operand order elsewhere. No, you are

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-18 Thread Uros Bizjak
On Tue, Feb 18, 2014 at 11:06 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Please don't change srcp pattern, it should be defined similar to vrcpss (aka sse_vmrcpv4sf). You need to switch operand order elsewhere. No, you are correct. Operands should be swapped as in your patch.

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-17 Thread Kirill Yukhin
Hello Uroš, On 13 Feb 18:25, Uros Bizjak wrote: On Thu, Feb 13, 2014 at 1:55 PM, Uros Bizjak ubiz...@gmail.com wrote: Please don't change srcp pattern, it should be defined similar to vrcpss (aka sse_vmrcpv4sf). You need to switch operand order elsewhere. No, you are correct.

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-17 Thread Uros Bizjak
On Mon, Feb 17, 2014 at 1:26 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Please don't change srcp pattern, it should be defined similar to vrcpss (aka sse_vmrcpv4sf). You need to switch operand order elsewhere. No, you are correct. Operands should be swapped as in your patch. Eh,

[PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-13 Thread Kirill Yukhin
Hello, I’ve noticed that _mm512_permutexvar_epi[64|32] intrinsics have wrong arguments order. As per [1] first argument is index. For vmpermps/vpermpd intrinsics are fine, but I’ve changed tests to call CALC with same arg order as intrinsic. here is the same problem (wrong argument order) with

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-13 Thread Uros Bizjak
On Thu, Feb 13, 2014 at 11:44 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: I've noticed that _mm512_permutexvar_epi[64|32] intrinsics have wrong arguments order. As per [1] first argument is index. For vmpermps/vpermpd intrinsics are fine, but I've changed tests to call CALC with same arg

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-13 Thread Uros Bizjak
On Thu, Feb 13, 2014 at 1:37 PM, Uros Bizjak ubiz...@gmail.com wrote: I've noticed that _mm512_permutexvar_epi[64|32] intrinsics have wrong arguments order. As per [1] first argument is index. For vmpermps/vpermpd intrinsics are fine, but I've changed tests to call CALC with same arg order as

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-13 Thread Uros Bizjak
On Thu, Feb 13, 2014 at 1:55 PM, Uros Bizjak ubiz...@gmail.com wrote: I've noticed that _mm512_permutexvar_epi[64|32] intrinsics have wrong arguments order. As per [1] first argument is index. For vmpermps/vpermpd intrinsics are fine, but I've changed tests to call CALC with same arg order as