Re: [PATCH, AArch64] Fix for PR61202

2014-05-21 Thread Marcus Shawcroft
On 20 May 2014 18:37, Carrot Wei car...@google.com wrote: Hi James Thank you for pointing this out. In the new patch I removed the modification of vqdmulh_n_s32 and vqdmulhq_n_s32. Passed dejagnu testing on aarch64 qemu again. OK for trunk, 4.9 and 4.8? 2014-05-20 Guozhi Wei

Re: [PATCH, AArch64] Fix for PR61202

2014-05-21 Thread Marcus Shawcroft
On 21 May 2014 09:28, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 20 May 2014 18:37, Carrot Wei car...@google.com wrote: Hi James Thank you for pointing this out. In the new patch I removed the modification of vqdmulh_n_s32 and vqdmulhq_n_s32. Passed dejagnu testing on aarch64

Re: [PATCH, AArch64] Fix for PR61202

2014-05-21 Thread Richard Biener
On Wed, 21 May 2014, Marcus Shawcroft wrote: On 21 May 2014 09:28, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 20 May 2014 18:37, Carrot Wei car...@google.com wrote: Hi James Thank you for pointing this out. In the new patch I removed the modification of vqdmulh_n_s32 and

Re: [PATCH, AArch64] Fix for PR61202

2014-05-21 Thread Carrot Wei
Committed to trunk, 4.9, and waiting for the release of 4.8.3. OK for google/main and google/4.9? thanks Carrot On Wed, May 21, 2014 at 1:34 AM, Richard Biener rguent...@suse.de wrote: On Wed, 21 May 2014, Marcus Shawcroft wrote: On 21 May 2014 09:28, Marcus Shawcroft

Re: [PATCH, AArch64] Fix for PR61202

2014-05-21 Thread Xinliang David Li
Since it is in trunk and gcc-4_9-branch, there is no need to put into google branches -- will get it automatically after next merge. David On Wed, May 21, 2014 at 8:13 PM, Carrot Wei car...@google.com wrote: Committed to trunk, 4.9, and waiting for the release of 4.8.3. OK for google/main and

Re: [PATCH, AArch64] Fix for PR61202

2014-05-20 Thread James Greenhalgh
On Tue, May 20, 2014 at 07:18:40AM +0100, Carrot Wei wrote: Hi Hi, The last operand of instruction sqdmulh can only be low fp registers, so we should use constraint x. But the intrinsic functions use w. This patch fixed the constrains in these intrinsics. This restriction is only on the

Re: [PATCH, AArch64] Fix for PR61202

2014-05-20 Thread Carrot Wei
Hi James Thank you for pointing this out. In the new patch I removed the modification of vqdmulh_n_s32 and vqdmulhq_n_s32. Passed dejagnu testing on aarch64 qemu again. OK for trunk, 4.9 and 4.8? 2014-05-20 Guozhi Wei car...@google.com * config/aarch64/arm_neon.h (vqdmulh_n_s16):