On Sat, Jun 4, 2016 at 4:25 AM, kugan <kugan.vivekanandara...@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 <kug...@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 <kug...@linaro.org> > > PR middle-end/71281 > * g++.dg/torture/pr71281.C: New test.
