RE: [backport gcc10, gcc9] Requet to backport PR97969

[Przemyslaw Wirkus <Przemyslaw.Wirkus@arm.com>]

> -----Original Message-----
> From: Christophe Lyon <christophe.lyon@linaro.org>
> Sent: 03 June 2021 09:45
> To: Przemyslaw Wirkus <Przemyslaw.Wirkus@arm.com>
> Cc: Vladimir Makarov <vmakarov@redhat.com>; jakub@redhat.com; Richard
> Earnshaw <Richard.Earnshaw@arm.com>; Richard Biener
> <rguenther@suse.de>; gcc-patches@gcc.gnu.org; Ramana Radhakrishnan
> <Ramana.Radhakrishnan@arm.com>
> Subject: Re: [backport gcc10, gcc9] Requet to backport PR97969
> 
> On Thu, 3 Jun 2021 at 00:31, Przemyslaw Wirkus via Gcc-patches <gcc-
> patches@gcc.gnu.org> wrote:
> >
> > Hi,
> >
> > > -----Original Message-----
> > > From: Vladimir Makarov <vmakarov@redhat.com>
> > > Sent: 31 May 2021 16:52
> > > To: Przemyslaw Wirkus <Przemyslaw.Wirkus@arm.com>; Richard Biener
> > > <rguenther@suse.de>
> > > Cc: gcc-patches@gcc.gnu.org; jakub@redhat.com; nickc@redhat.com;
> > > Richard Earnshaw <Richard.Earnshaw@arm.com>; Ramana
> Radhakrishnan
> > > <Ramana.Radhakrishnan@arm.com>; Kyrylo Tkachov
> > > <Kyrylo.Tkachov@arm.com>
> > > Subject: Re: [backport gcc10, gcc9] Requet to backport PR97969
> > >
> > >
> > > On 2021-05-25 5:14 a.m., Przemyslaw Wirkus wrote:
> > > > Hi,
> > > > Just a follow up after GCC 11 release.
> > > >
> > > > I've backported to gcc-10 branch (without any change to original
> > > > patches)
> > > > PR97969 and following PR98722 & PR98777 patches.
> > > >
> > > > Commits apply cleanly without changes.
> > > > Built and regression tested on:
> > > > * arm-none-eabi and
> > > > * aarch64-none-linux-gnu cross toolchains.
> > > >
> > > > There were no issues and no regressions (all OK).
> > > >
> > > > OK for backport to gcc-10 branch ?
> > >
> > > Sorry for delay with the answer due to my vacation.
> > >
> > > As the patches did not introduce new PRs I believe they are ok for gcc-10.
> >
> > Backported to gcc-10 branch. Thank you for your support.
> >
> 
> Hi,
> 
> I'm surprised to see many new errors on arm after the backport for PR98722
> See: https://people.linaro.org/~christophe.lyon/cross-validation/gcc/gcc-
> 10/r10-9881-g1791b11d9cae388ae18a768eeb96c998439c986a/report-build-
> info.html
> 
> Przemyslaw, Vladimir do you confirm r10-9881 has no such errors (new
> ICEs) on your side?

Apologies.

I've built and regtested before submitting backport yesterday.
I will check on my side and build one of your failing configurations.

Przemyslaw

> Thanks
> 
> > Kind regards
> > Przemyslaw
> >
> > > Thank you.
> > >
> > > >
> > > > Kind regards,
> > > > Przemyslaw Wirkus
> > > >
> > > > ---
> > > > commits I've backported:
> > > >
> > > > commit cf2ac1c30af0fa783c8d72e527904dda5d8cc330
> > > > Author: Vladimir N. Makarov <vmakarov@redhat.com>
> > > > Date:   Tue Jan 12 11:26:15 2021 -0500
> > > >
> > > >      [PR97969] LRA: Transform pattern `plus (plus (hard reg,
> > > > const), pseudo)`
> > > after elimination
> > > >
> > > > commit 4334b524274203125193a08a8485250c41c2daa9
> > > > Author: Vladimir N. Makarov <vmakarov@redhat.com>
> > > > Date:   Wed Jan 20 11:40:14 2021 -0500
> > > >
> > > >      [PR98722] LRA: Check that target has no 3-op add insn to
> > > > transform 2
> > > plus expression.
> > > >
> > > > commit 68ba1039c7daf0485b167fe199ed7e8031158091
> > > > Author: Vladimir N. Makarov <vmakarov@redhat.com>
> > > > Date:   Thu Jan 21 17:27:01 2021 -0500
> > > >
> > > >      [PR98777] LRA: Use preliminary created pseudo for in LRA
> > > > elimination
> > > subpass
> > > >
> > > > $ ./contrib/git-backport.py
> > > > cf2ac1c30af0fa783c8d72e527904dda5d8cc330
> > > > $ ./contrib/git-backport.py
> > > > 4334b524274203125193a08a8485250c41c2daa9
> > > > $ ./contrib/git-backport.py
> > > > 68ba1039c7daf0485b167fe199ed7e8031158091
> > > >
> > > >
> > > >> Richard.
> >