This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR71281] ICE on gcc trunk on knl, wsm, ivb and bdw targets
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: kugan <kugan dot vivekanandarajah at linaro dot org>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 6 Jun 2016 10:54:24 +0200
- Subject: Re: [PR71281] ICE on gcc trunk on knl, wsm, ivb and bdw targets
- Authentication-results: sourceware.org; auth=none
- References: <57523C01 dot 5020808 at linaro dot org>
On Sat, Jun 4, 2016 at 4:25 AM, kugan <kugan.vivekanandarajah@linaro.org> wrote:
> Hi,
>
> PR71281 happens when we use factored out negate stmt in other
> reassociations. Since we don't set the uid for this stmt, we hit the
> gcc_assert (in reassoc_stmt_dominates_stmt_p) which checks for uid being
> set. Attached patch fixes this.
>
> Regression tested on x86-64-linux-gnu with no new regression. Is this OK for
> trunk?
Ok.
Thanks,
Richard.
> Thanks,
> Kugan
>
> gcc/ChangeLog:
>
> 2016-06-04 Kugan Vivekanandarajah <kuganv@linaro.org>
>
> PR middle-end/71281
> * tree-ssa-reassoc.c (reassociate_bb): Set uid for negate stmt.
>
>
> gcc/testsuite/ChangeLog:
>
> 2016-06-04 Kugan Vivekanandarajah <kuganv@linaro.org>
>
> PR middle-end/71281
> * g++.dg/torture/pr71281.C: New test.