On Fri, Jun 10, 2016 at 4:19 AM, kugan
<kugan.vivekanandara...@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  <kug...@linaro.org>
>
>         * gcc.dg/pr71478.c: New test.
>
> gcc/ChangeLog:
>
> 2016-06-10  Kugan Vivekanandarajah  <kug...@linaro.org>
>
>         * tree-ssa-reassoc.c (reassociate_bb): Remove (-1) from ops list for
>         vector integer type.

Reply via email to