This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Bug tree-optimization/34649] redundant if expression in find_conditional_asserts
- From: "Richard Guenther" <richard dot guenther at gmail dot com>
- To: "Uros Bizjak" <ubizjak at gmail dot com>
- Cc: "GCC Patches" <gcc-patches at gcc dot gnu dot org>, "zhouyi zhou" <zhouyi04 at ios dot cn>
- Date: Fri, 4 Jan 2008 14:05:13 +0100
- Subject: Re: [Bug tree-optimization/34649] redundant if expression in find_conditional_asserts
- References: <477E2C79.5050307@gmail.com>
On Jan 4, 2008 1:54 PM, Uros Bizjak <ubizjak@gmail.com> wrote:
> Hello!
>
> > Please provide a regression test for me, it is the first time I have a
> > chance to patch gcc, Thanks a lot :)
> [ Please look at http://gcc.gnu.org/contribute.html#testing ]
>
> The patch was bootstrapped and regression tested on x86_64-linux {,
> -m32} without new failures. But the middle-end maintainer should approve
> your commit.
>
> > ChangeLog Entry
> > 2008-01-04 Zhouyi Zhou <zhouzhouyi@FreeBSD.org>
> >
> > * tree-vrp.c (find_conditional_asserts): redundant if expression
> >
>
> Maybe "Remove redundant check that edge does not point to current bb
> before changing need_assert." is more informative here?
Ok with that change.
Richard.