Re: [PATCH ARM] Fix PR target/63209

2014-09-10 Thread Xinliang David Li
Fixed the formatting and committed it to trunk: r215136. Will backport it to gcc-4_9 branch. thanks, David On Wed, Sep 10, 2014 at 11:24 AM, Richard Earnshaw wrote: > On 09/09/14 19:45, Xinliang David Li wrote: >> Richard, thanks for the review. The revised patch is attached. Is this >> one OK

Re: [PATCH ARM] Fix PR target/63209

2014-09-10 Thread Richard Earnshaw
On 09/09/14 19:45, Xinliang David Li wrote: > Richard, thanks for the review. The revised patch is attached. Is this > one OK (after testing is done)? > > David > > OK, but ... > 2014-09-08 Xinliang David Li > > PR target/63209 > * config/arm/arm.md (movcond_addsi): Handle

Re: [PATCH ARM] Fix PR target/63209

2014-09-10 Thread Xinliang David Li
With gcc regression test, no regressions are found. David On Tue, Sep 9, 2014 at 11:45 AM, Xinliang David Li wrote: > Richard, thanks for the review. The revised patch is attached. Is this > one OK (after testing is done)? > > David > > > 2014-09-08 Xinliang David Li > > PR target/632

Re: [PATCH ARM] Fix PR target/63209

2014-09-09 Thread Xinliang David Li
Richard, thanks for the review. The revised patch is attached. Is this one OK (after testing is done)? David 2014-09-08 Xinliang David Li PR target/63209 * config/arm/arm.md (movcond_addsi): Handle case where source and target operands are the same 2014-09-08 Xinlia

Re: [PATCH ARM] Fix PR target/63209

2014-09-09 Thread Richard Earnshaw
On 09/09/14 16:58, Xinliang David Li wrote: > The attached patch fixed the problem reported in PR63209. The problem > exists in both gcc4.9 and trunk. > > Regression test on arm-unknown-linux-gnueabi passes. > > Ok for gcc-4.9 and trunk? > No, this isn't right. I don't think you need a new pat

[PATCH ARM] Fix PR target/63209

2014-09-09 Thread Xinliang David Li
The attached patch fixed the problem reported in PR63209. The problem exists in both gcc4.9 and trunk. Regression test on arm-unknown-linux-gnueabi passes. Ok for gcc-4.9 and trunk? (I sent the patch last night, but it got lost somehow) David Index: ChangeLog ==