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

[Jiong Wang]

On 26/09/14 16:05, H.J. Lu wrote:
On Thu, Sep 25, 2014 at 9:43 AM, Jiong Wang <jiong.w...@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.