On Thu, 3 Jun 2021 at 10:54, Przemyslaw Wirkus <przemyslaw.wir...@arm.com> wrote: > > > > > -----Original Message----- > > From: Christophe Lyon <christophe.l...@linaro.org> > > Sent: 03 June 2021 09:45 > > To: Przemyslaw Wirkus <przemyslaw.wir...@arm.com> > > Cc: Vladimir Makarov <vmaka...@redhat.com>; ja...@redhat.com; Richard > > Earnshaw <richard.earns...@arm.com>; Richard Biener > > <rguent...@suse.de>; gcc-patches@gcc.gnu.org; Ramana Radhakrishnan > > <ramana.radhakrish...@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- > > patc...@gcc.gnu.org> wrote: > > > > > > Hi, > > > > > > > -----Original Message----- > > > > From: Vladimir Makarov <vmaka...@redhat.com> > > > > Sent: 31 May 2021 16:52 > > > > To: Przemyslaw Wirkus <przemyslaw.wir...@arm.com>; Richard Biener > > > > <rguent...@suse.de> > > > > Cc: gcc-patches@gcc.gnu.org; ja...@redhat.com; ni...@redhat.com; > > > > Richard Earnshaw <richard.earns...@arm.com>; Ramana > > Radhakrishnan > > > > <ramana.radhakrish...@arm.com>; Kyrylo Tkachov > > > > <kyrylo.tkac...@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. >
After I sent the previous email, I received the validation results for the next backport, and it seems it fixes the ICEs introduced by r10-9881: https://people.linaro.org/~christophe.lyon/cross-validation/gcc/gcc-10/r10-9882-g05f6971ac40912ef062915f88b3ea0bf27278285/report-build-info.html I guess you ran validations with the 3 backports combined, rather than individually? So it looks OK now. Thanks, Christophe > Przemyslaw > > > Thanks > > > > > Kind regards > > > Przemyslaw > > > > > > > Thank you. > > > > > > > > > > > > > > Kind regards, > > > > > Przemyslaw Wirkus > > > > > > > > > > --- > > > > > commits I've backported: > > > > > > > > > > commit cf2ac1c30af0fa783c8d72e527904dda5d8cc330 > > > > > Author: Vladimir N. Makarov <vmaka...@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 <vmaka...@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 <vmaka...@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. > > >
