CCing release maintainers as well as they need to approve this backport if Vlad is happy with it.
Vlad - is this ok to go back as it fixes a bug for ARM in the 4.9 tree that came up in building bits of debian. Ramana On Mon, Dec 1, 2014 at 5:24 PM, Renlin Li <renlin...@arm.com> wrote: > On 01/12/14 15:58, H.J. Lu wrote: >> >> On Thu, Nov 27, 2014 at 8:38 AM, Renlin Li <renlin...@arm.com> wrote: >>> >>> On 27/11/14 15:37, H.J. Lu wrote: >>>> >>>> On Thu, Nov 27, 2014 at 7:32 AM, Renlin Li <renlin...@arm.com> wrote: >>>>> >>>>> On 26/11/14 18:12, H.J. Lu wrote: >>>>>> >>>>>> On Wed, Nov 26, 2014 at 10:09 AM, Renlin Li <renlin...@arm.com> wrote: >>>>>>> >>>>>>> On 26/11/14 12:16, H.J. Lu wrote: >>>>>>>> >>>>>>>> On Wed, Nov 26, 2014 at 4:07 AM, Renlin Li <renlin...@arm.com> >>>>>>>> wrote: >>>>>>>>> >>>>>>>>> On 20/11/14 16:17, Renlin Li wrote: >>>>>>>>>> >>>>>>>>>> Hi all, >>>>>>>>>> >>>>>>>>>> This is a backport for gcc-4_9-branch of the patch "[PR63762]GCC >>>>>>>>>> generates >>>>>>>>>> UNPREDICTABLE STR with Rn = Rt for arm" posted in: >>>>>>>>>> https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02253.html >>>>>>>>>> >>>>>>>>>> arm-none-eabi has been test on the model, no new issues. >>>>>>>>>> bootstrapping >>>>>>>>>> and >>>>>>>>>> regression tested on x86, no new issues. >>>>>>>>>> >>>>>>>>>> Is it Okay for gcc-4_9-branch? >>>>>>>>>> >>>>>>>>>> gcc/ChangeLog: >>>>>>>>>> >>>>>>>>>> 2014-11-20 Renlin Li <renlin...@arm.com> >>>>>>>>>> >>>>>>>>>> PR middle-end/63762 >>>>>>>>>> * ira.c (ira): Update preferred class. >>>>>>>>>> >>>>>>>>>> gcc/testsuite/ChangeLog: >>>>>>>>>> >>>>>>>>>> 2014-11-20 Renlin Li <renlin...@arm.com> >>>>>>>>>> >>>>>>>>>> PR middle-end/63762 >>>>>>>>>> * gcc.dg/pr63762.c: New. >>>>>>>>> >>>>>>>>> Ping for it. >>>>>>>>> >>>>>>>> Please verify if it is the real fix for >>>>>>>> >>>>>>>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63661 >>>>>>>> >>>>>>>> If yes, please add a testcase for PR 63661 and mention it in >>>>>>>> your ChangeLog entry. >>>>>>>> >>>>>>>> Thanks. >>>>>>>> >>>>>>>> >>>>>>> Hi H.J. >>>>>>> >>>>>>> Yes, I have verified that, this patch additionally fixes PR 63661. >>>>>>> >>>>>>> I observed the same behaviour as I saw on arm backend. It will be >>>>>>> great >>>>>>> if >>>>>>> you can double check they are caused by exactly the same reason. >>>>>> >>>>>> I will ask our people to take a look. >>>>>> >>>>>>> A new testcase has been added, ChangeLog has been updated to reflect >>>>>>> the >>>>>>> change. Updated patch has bee attached. >>>>>>> Okay for gcc-4_9-branch? >>>>>>> >>>>>>> Regards, >>>>>>> Renlin Li >>>>>>> >>>>>>> >>>>>>> gcc/ChangeLog: >>>>>>> >>>>>>> 2014-11-26 Renlin Li <renlin...@arm.com> >>>>>>> >>>>>>> PR middle-end/63762 >>>>>>> PR middle-end/63661 >>>>>>> * ira.c (ira): Update preferred class. >>>>>>> >>>>>>> gcc/testsuite/ChangeLog: >>>>>>> >>>>>>> 2014-11-26 Renlin Li <renlin...@arm.com> >>>>>>> >>>>>>> PR middle-end/63661 >>>>>>> PR middle-end/63762 >>>>>>> * testsuite/gcc.dg/pr63661.c: New. >>>>>>> * testsuite/gcc.dg/pr63762.c: New. >>>>>>> >>>>>>> >>>>>> pr63661.c should be moved to gcc.target/i386 and run it >>>>>> on PIC target. >>>>>> >>>>>> Thanks. >>>>>> >>>>> Hi H.J. >>>>> >>>>> The patch has been adjusted according to your suggestion. >>>>> >>>>> >>>>> gcc/ChangeLog: >>>>> >>>>> 2014-11-27 Renlin Li <renlin...@arm.com> >>>>> >>>>> PR middle-end/63762 >>>>> PR target/63661 >>>>> * ira.c (ira): Update preferred class. >>>>> >>>>> gcc/testsuite/ChangeLog: >>>>> >>>>> 2014-11-27 Renlin Li <renlin...@arm.com> >>>>> >>>>> PR middle-end/63762 >>>>> PR target/63661 >>>>> * testsuite/gcc.dg/pr63762.c: New. >>>>> * testsuite/gcc.target/i386/pr63661.c: New. >>>> >>>> ^^^^^^^^^^^ No testsuite/ >>> >>> >>> Hi H.J. >>>> >>>> gcc.target/i386/pr63661.c should be checked into trunk first. >>>> >>> A separate patch is sent to mailing list for this. >>> >>> >>> ChangeLog has been corrected. >>> >>> Regards, >>> Renlin Li >>> >>> gcc/ChangeLog: >>> >>> 2014-11-27 Renlin Li <renlin...@arm.com> >>> >>> PR middle-end/63762 >>> PR target/63661 >>> * ira.c (ira): Update preferred class. >>> >>> gcc/testsuite/ChangeLog: >>> >>> 2014-11-27 Renlin Li <renlin...@arm.com> >>> >>> PR middle-end/63762 >>> PR target/63661 >>> * gcc.dg/pr63762.c: New. >>> * gcc.target/i386/pr63661.c: New. >>> >>> >> You need to update gcc.target/i386/pr63661.c from trunk. >> >> > > Update the test case for pr63661 from trunk. > > Okay for gcc-4_9-branch? > > gcc/ChangeLog: > > 2014-12-01 Renlin Li <renlin...@arm.com> > > PR middle-end/63762 > PR target/63661 > * ira.c (ira): Update preferred class. > > gcc/testsuite/ChangeLog: > > 2014-12-01 Renlin Li <renlin...@arm.com> > H.J Lu <hongjiu...@intel.com> > > > PR middle-end/63762 > PR target/63661 > * gcc.dg/pr63762.c: New. > * gcc.target/i386/pr63661.c: New.
