On Fri, Apr 29, 2016 at 6:04 PM, Bin Cheng <bin.ch...@arm.com> wrote: > Hi, > This patch fixes a latent bug in IVOPT. Variable "depends_on" should be > checked before recording invariant expression, otherwise we end up with > constant (even ZERO) invariant expressions. Apparently this results in wrong > register pressure. > > Bootstrap and test on x86_64. Is it OK?
Ok. Richard. > Thanks, > bin > > 2016-04-28 Bin Cheng <bin.ch...@arm.com> > > * tree-ssa-loop-ivopts.c (get_computation_cost_at): Check depends_on > before using it.