This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR71478] Fix ICE in tree-ssa-reassoc.c
- 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: Fri, 10 Jun 2016 13:34:07 +0200
- Subject: Re: [PR71478] Fix ICE in tree-ssa-reassoc.c
- Authentication-results: sourceware.org; auth=none
- References: <575A23AD dot 3040204 at linaro dot org>
On Fri, Jun 10, 2016 at 4:19 AM, kugan
<kugan.vivekanandarajah@linaro.org> wrote:
> Hi,
>
> In PR71478, for vector negation of ssa produced by call stmt, we add vector
> (-1) and ssa to the ops list. However, in the place where we remove the (-1)
> from ops list, we failed to do this for vector integer. As a result,
> rewrite_expr_tree wrongly assumes that it is working with gimple_assign.
>
> Attached patch fixes the place where we remove the vector (-1).
>
> Regression tested on x86-64-linux-gnu with no new regressions. Regression
> testing on aarc64-linux-gnu is ongoing. Is this OK for trunk?
Ok.
Richard.
> Thanks,
> Kugan
>
> gcc/testsuite/ChangeLog:
>
> 2016-06-10 Kugan Vivekanandarajah <kuganv@linaro.org>
>
> * gcc.dg/pr71478.c: New test.
>
> gcc/ChangeLog:
>
> 2016-06-10 Kugan Vivekanandarajah <kuganv@linaro.org>
>
> * tree-ssa-reassoc.c (reassociate_bb): Remove (-1) from ops list for
> vector integer type.