This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [vtab and trunk?] ifcvt: try 'then' only if 'else' is unsuitable
- From: "Richard Guenther" <richard dot guenther at gmail dot com>
- To: "Alexandre Oliva" <aoliva at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sat, 1 Mar 2008 13:16:33 +0100
- Subject: Re: [vtab and trunk?] ifcvt: try 'then' only if 'else' is unsuitable
- References: <ork5q6a8xm.fsf@free.oliva.athome.lsd.ic.unicamp.br> <ord4txh5fq.fsf@oliva.athome.lsd.ic.unicamp.br> <84fc9c000711260205r1cb23c03v3bd6ef3e265149ff@mail.gmail.com> <or1w6vyuj3.fsf@oliva.athome.lsd.ic.unicamp.br>
On Sat, Mar 1, 2008 at 4:48 AM, Alexandre Oliva <aoliva@redhat.com> wrote:
>
> On Nov 26, 2007, "Richard Guenther" <richard.guenther@gmail.com> wrote:
>
> > On Nov 26, 2007 7:44 AM, Alexandre Oliva <aoliva@redhat.com> wrote:
> >> On Oct 1, 2007, Alexandre Oliva <aoliva@redhat.com> wrote:
>
> >> > Index: gcc/ChangeLog
> >> > from Alexandre Oliva <aoliva@redhat.com>
>
> >> > * ifcvt.c (noce_process_if_block): Try to handle only the then
> >> > block if the else block exists but isn't suitable.
>
> >> :ADDPATCH ifcvt:
>
> >> Ping?
>
> >> http://gcc.gnu.org/ml/gcc-patches/2007-10/msg00070.html
>
> > First this patch misses a testcase, 2nd, as it doesn't look like a regression
> > fix (thouhg it looks straight-forward), it needs to wait for stage1.
>
> It's stage1 now, and I don't have a testcase. The only one I had
> involved debug insns. Do I really need to come up with one that
> doesn't, or can this go in as it is at this time?
>
> I've re-tested on x86_64-linux-gnu, FWIW.
Hm, well. Ok anyway then.
Thanks,
Richard.