This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: patch to fix PR82353
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 12 Oct 2017 19:11:14 +0200
- Subject: Re: patch to fix PR82353
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx05.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx05.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=jakub at redhat dot com
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 87E1290C9F
- References: <e055dbd4-34fd-7665-3eed-42c41cb27b5d@redhat.com> <20171011211106.GQ14653@tucnak> <b2fcbd95-580a-7d7c-9888-fcb97c13058d@redhat.com> <20171012164921.GY14653@tucnak> <0700d384-c580-efd3-e1b5-f7aac40d92c9@redhat.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Oct 12, 2017 at 01:05:21PM -0400, Vladimir Makarov wrote:
>
>
> On 10/12/2017 12:49 PM, Jakub Jelinek wrote:
> > Hi!
> >
> > On Wed, Oct 11, 2017 at 06:41:05PM -0400, Vladimir Makarov wrote:
> > > > Tested on x86_64-linux -m32/-m64, and verified with cc1plus before your
> > > > change, ok for trunk?
> > BTW, I think it is quite fragile to scan for the reload messages, so I've
> > cooked up a runtime test that fails before your patch and succeeds with your
> > patch. Tested on x86_64-linux with -m32/-m64 (both with your patch reverted
> > and without), ok for trunk?
> >
> >
> OK.
Thanks.
> FYI, I am going to revert LRA part of the patch because it results in a
> failure bootstrap with go or ada. I guess the new version of the patch will
> be ready tomorrow or on Monday.
Yeah, I've just noticed that too (my bootstrap failed in Ada). I'll defer
committing the patch after you commit your new fix then.
Jakub