This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: cosmetic change - simplify cse.c:preferable()
- From: Richard Guenther <rguenther at suse dot de>
- To: Dimitrios Apostolou <jimis at gmx dot net>
- Cc: Dodji Seketeli <dodji at seketeli dot org>, gcc-patches at gcc dot gnu dot org, Andrey Belevantsev <abel at ispras dot ru>, Richard Henderson <rth at redhat dot com>, christophe dot jaillet at wanadoo dot fr
- Date: Mon, 6 Aug 2012 13:35:42 +0200 (CEST)
- Subject: Re: cosmetic change - simplify cse.c:preferable()
- References: <alpine.LNX.2.02.1207081038340.4288@localhost.localdomain> <m3bojcq2t2.fsf@seketeli.org> <alpine.LNX.2.00.1207191038200.17088@jbgna.fhfr.qr> <alpine.LNX.2.02.1208040007420.20463@localhost.localdomain>
On Sat, 4 Aug 2012, Dimitrios Apostolou wrote:
> On Thu, 19 Jul 2012, Richard Guenther wrote:
> >
> > I don't think it's any good or clearer to understand.
>
> Hi Richi, I had forgotten I prepared this for PR #19832, maybe you want to
> take a look. FWIW, with my patch applied there is a difference of ~3 M instr,
> which is almost unmeasurable in time. But we can close the PR even with the
> simple patch Cristophe posted, since mine does not make things clearer.
Christophes patch is ok. We still want to if-convert this in the
compiler though.
Thanks,
Richard.