Re: [PATCH][AArch64] Vector shift by 64 fix
On 6 January 2014 11:52, Alex Velenko alex.vele...@arm.com wrote: Hi, This patch fixes vector shift by 64 behavior to meet reference manual expectations. Testcase included to check that expectations are now met. No regressions found. Is patch OK? OK /Marcus
Re: [PATCH][AArch64] Vector shift by 64 fix
Hi, Could someone, please, commit this patch, as I do not have permissions to do so. Kind regards, Alex On 23/01/14 12:04, Marcus Shawcroft wrote: On 6 January 2014 11:52, Alex Velenko alex.vele...@arm.com wrote: Hi, This patch fixes vector shift by 64 behavior to meet reference manual expectations. Testcase included to check that expectations are now met. No regressions found. Is patch OK? OK /Marcus
Re: [PATCH][AArch64] Vector shift by 64 fix
On Thu, Jan 23, 2014 at 03:57:33PM +, Alex Velenko wrote: Hi, Could someone, please, commit this patch, as I do not have permissions to do so. Please don't top-post on this list. On 23/01/14 12:04, Marcus Shawcroft wrote: On 6 January 2014 11:52, Alex Velenko alex.vele...@arm.com wrote: Hi, This patch fixes vector shift by 64 behavior to meet reference manual expectations. Testcase included to check that expectations are now met. No regressions found. Is patch OK? OK /Marcus I've committed this on Alex' behalf as revision 206978. Thanks, James
Re: [PATCH][AArch64] Vector shift by 64 fix
On 06/01/14 11:52, Alex Velenko wrote: Hi, This patch fixes vector shift by 64 behavior to meet reference manual expectations. Testcase included to check that expectations are now met. No regressions found. Is patch OK? Thanks, Alex 2014-01-06 Alex Velenko alex.vele...@arm.com gcc/ * config/aarch64/aarch64-simd-builtins.def (ashr): DI mode removed. (ashr_simd): New builtin handling DI mode. * config/aarch64/aarch64-simd.md (aarch64_ashr_simddi): New pattern. (aarch64_sshr_simddi): New match pattern. * config/aarch64/arm_neon.h (vshr_n_s32): Builtin call modified. (vshrd_n_s64): Likewise. * config/aarch64/predicates.md (aarch64_shift_imm64_di): New predicate. gcc/testsuite/ * gcc.target/aarch64/sshr64_1.c: New testcase. Ping! Hi, Can someone, please, review the patch. Kind regards, Alex