This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Improve switchconv optimization (PR tree-optimization/79472)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Richard Biener <rguenther at suse dot de>
- Cc: Martin Jambor <mjambor at suse dot cz>, gcc-patches at gcc dot gnu dot org
- Date: Tue, 2 May 2017 14:12:40 +0200
- Subject: Re: [PATCH] Improve switchconv optimization (PR tree-optimization/79472)
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx04.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx04.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=jakub at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 9FD11F4B71
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 9FD11F4B71
- References: <20170214200445.GY1849@tucnak> <20DAA120-6749-4D13-8C8D-A91EC067CACC@suse.de> <20170215071721.GG1849@tucnak> <84e063a8-5cd7-5f99-0457-33c80cf040c7@suse.de> <20170215115130.GI1849@tucnak> <20170429163055.GR1809@tucnak> <alpine.LSU.2.20.1705021315470.17885@zhemvz.fhfr.qr>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, May 02, 2017 at 01:16:05PM +0200, Richard Biener wrote:
> On Sat, 29 Apr 2017, Jakub Jelinek wrote:
>
> > On Wed, Feb 15, 2017 at 12:51:30PM +0100, Jakub Jelinek wrote:
> > > On Wed, Feb 15, 2017 at 12:46:44PM +0100, Richard Biener wrote:
> > > > >> Possibly, but for GCC 8.
> > > > >
> > > > > To both this switchconv patch and the potential improvement for loading
> > > > > from const arrays (can create an enhancement PR for that), or just the
> > > > > latter?
> > > >
> > > > Both I think - the patch is pretty big.
> > >
> > > Ok, I'll queue the patch for GCC8 then.
> >
> > If the tree-vrp.c change makes it in, is this patch ok for trunk too now
> > that we are in stage1? Bootstrapped/regtested on x86_64-linux and
> > i686-linux on top of the tree-vrp.c change (without the tree-vrp.c change
> > vrp40.c regresses).
>
> Ok.
So with XFAILing vrp40.c for now until the constant load opt is resolved?
Jakub