On Fri, 11 Mar 2016, Sebastian Pop wrote: > On Fri, Mar 11, 2016 at 9:14 AM, Tom de Vries <tom_devr...@mentor.com> > wrote: > > > Hi, > > > > this patch fixes PR70045, a graphite 6 regression. > > > > The problem is as follows: in graphite_create_new_loop_guard, a condition > > cond_expr is constructed using an upper bound expression *ub. > > > > During the call: > > ... > > exit_edge = create_empty_if_region_on_edge (entry_edge, cond_expr); > > ... > > the cond_expr is modified in place, which has as side-effect that *ub is > > modified. > > > > The patch fixes this by unsharing the cond_expr before passing it as > > argument to create_empty_if_region_on_edge. > > > > Bootstrapped and reg-tested on x86_64. > > > > OK for stage4 trunk? > > > > > Thanks Tom, the patch looks good to me. > I will let Richi decide whether it is still appropriate to commit the > change to trunk.
As it is a regression it's fine. Richard.
