Re: [COMMITTED][PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

[Jiong Wang <jiong dot wang at arm dot com>]

On 26/09/14 16:05, H.J. Lu wrote:
> On Thu, Sep 25, 2014 at 9:43 AM, Jiong Wang <jiong.wang@arm.com> wrote:
> > On 25/09/14 17:24, Jeff Law wrote:
> > > On 09/25/14 09:04, Jiong Wang wrote:
> > > > new patch updated.
> > > >
> > > > pass bootstrap and no regression, both check-gcc and check-g++, on the
> > > > x86.
> > > >
> > > > OK for trunk?
> > > >
> > > > thanks.
> > > >
> > > > gcc/
> > > >       * shrink-wrap.c (move_insn_for_shrink_wrap): Initialize the live-in
> > > > of
> > > >       new created BB as the intersection of live-in from "old_dest" and
> > > > live-out
> > > >       from "bb".
> > > Please include a ChangeLog entry for the testsuite.  Something like:
> > >
> > >          * gcc.target/i386/shrink_wrap_1.c: New test.
> This fails on Linux/x86 and with -m32 on Linux/x86-64.
sorry, my test machine is x86-64, I think the shrink wrap test itself is 
very fragile because
it's highly related insn generated.

could you mark that testcase using something like 
"dg-require-effective-target lp64"?

> > > With that addition, OK for the trunk.
> >
> > committed as r215611.