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.