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 wrote: > Committed to trunk, 4.9, and waiting for the release of 4.8.3. > > OK for google/main and google/4.9? >

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 wrote: > On Wed, 21 May 2014, Marcus Shawcroft wrote: > >> On 21 May 2014 09:28, Marcus Shawcroft wrote: >> > On 20 May 2014 18:37, Ca

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 wrote: > > On 20 May 2014 18:37, Carrot Wei 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

Re: [PATCH, AArch64] Fix for PR61202

2014-05-21 Thread Marcus Shawcroft
On 21 May 2014 09:28, Marcus Shawcroft wrote: > On 20 May 2014 18:37, Carrot Wei 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

Re: [PATCH, AArch64] Fix for PR61202

2014-05-21 Thread Marcus Shawcroft
On 20 May 2014 18:37, Carrot Wei 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 > > * config/

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 * config/aarch64/arm_neon.h (vqdmulh_n_s16): Change the la

Re: [PATCH, AArch64] Fix for PR61202

2014-05-19 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

[PATCH, AArch64] Fix for PR61202

2014-05-19 Thread Carrot Wei
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. Passed dejagnu test on aarch64 qemu. OK for trunk, 4.9 and 4.8? thanks Guozhi Wei 2014-05-19 Guoz