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

2014-09-26 Thread H.J. Lu
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/ *

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

2014-09-26 Thread 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?

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

2014-09-26 Thread H.J. Lu
On Fri, Sep 26, 2014 at 8:14 AM, Jiong Wang jiong.w...@arm.com wrote: 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

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

2014-09-26 Thread Jiong Wang
On 26/09/14 16:28, H.J. Lu wrote: On Fri, Sep 26, 2014 at 8:14 AM, Jiong Wang jiong.w...@arm.com wrote: could you mark that testcase using something like dg-require-effective-target lp64? I checked in this patch to skip it on ia32. great, thanks ! -- Jiong

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

2014-09-25 Thread Zhenqiang Chen
-Original Message- From: Jiong Wang [mailto:jiong.w...@arm.com] Sent: Thursday, September 25, 2014 2:13 AM To: Jeff Law; Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge On 22/09/14 18:51, Jeff Law

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

2014-09-25 Thread Jiong Wang
On 25/09/14 09:52, Zhenqiang Chen wrote: -Original Message- From: Jiong Wang [mailto:jiong.w...@arm.com] Sent: Thursday, September 25, 2014 2:13 AM To: Jeff Law; Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, 2/2] shrink wrap a function with a single loop: split

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

2014-09-25 Thread Jeff Law
a function with a single loop: split live_edge On 22/09/14 18:51, Jeff Law wrote: On 09/22/14 04:24, Jiong Wang wrote: Great. Can you send an updated patchkit for review. patch attached. please review, thanks. gcc/ * shrink-wrap.c (move_insn_for_shrink_wrap): Initialize the live-in of new

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

2014-09-25 Thread Jiong Wang
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

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

2014-09-24 Thread Jiong Wang
On 22/09/14 18:51, Jeff Law wrote: On 09/22/14 04:24, Jiong Wang wrote: Great. Can you send an updated patchkit for review. patch attached. please review, 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

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

2014-09-22 Thread Jiong Wang
On 19/09/14 17:19, Jeff Law wrote: On 09/19/14 10:02, Jiong Wang wrote: On 19/09/14 16:49, Jeff Law wrote: Probably. Though I'd be a bit concerned with next_block-next_bb. Wouldn't it be safer to stash away the relevant basic block prior to the call to split_edge, then use that saved copy.

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

2014-09-22 Thread Jeff Law
On 09/22/14 04:24, Jiong Wang wrote: Great. Can you send an updated patchkit for review. patch attached. please review, 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.

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

2014-09-19 Thread Jiong Wang
; Jeff Law Subject: Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge On 08/05/14 09:07, Zhenqiang Chen wrote: static bool move_insn_for_shrink_wrap (basic_block bb, rtx insn, const HARD_REG_SET uses

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

2014-09-19 Thread Jeff Law
To: Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org; Jeff Law Subject: Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge On 08/05/14 09:07, Zhenqiang Chen wrote: static bool move_insn_for_shrink_wrap (basic_block bb, rtx insn, const HARD_REG_SET

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

2014-09-19 Thread Jiong Wang
, September 15, 2014 11:28 PM To: Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org; Jeff Law Subject: Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge On 08/05/14 09:07, Zhenqiang Chen wrote: static bool move_insn_for_shrink_wrap (basic_block bb, rtx insn

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

2014-09-19 Thread Jeff Law
] On Behalf Of Jiong Wang Sent: Monday, September 15, 2014 11:28 PM To: Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org; Jeff Law Subject: Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge On 08/05/14 09:07, Zhenqiang Chen wrote: static bool move_insn_for_shrink_wrap

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

2014-09-18 Thread Jeff Law
] shrink wrap a function with a single loop: split live_edge On 08/05/14 09:07, Zhenqiang Chen wrote: static bool move_insn_for_shrink_wrap (basic_block bb, rtx insn, const HARD_REG_SET uses, const HARD_REG_SET defs

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

2014-09-16 Thread Zhenqiang Chen
-Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Jiong Wang Sent: Monday, September 15, 2014 11:28 PM To: Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org; Jeff Law Subject: Re: [PATCH, 2/2] shrink wrap a function with a single

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

2014-09-15 Thread Jiong Wang
On 08/05/14 09:07, Zhenqiang Chen wrote: static bool move_insn_for_shrink_wrap (basic_block bb, rtx insn, const HARD_REG_SET uses, const HARD_REG_SET defs, - HARD_REG_SET *last_uses) +

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

2014-05-15 Thread Zhenqiang Chen
On 15 May 2014 02:03, Jeff Law l...@redhat.com wrote: On 05/13/14 03:49, Zhenqiang Chen wrote: On 9 May 2014 14:08, Jeff Law l...@redhat.com wrote: On 05/08/14 02:07, Zhenqiang Chen wrote: Hi, The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry

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

2014-05-14 Thread Jeff Law
On 05/13/14 03:49, Zhenqiang Chen wrote: On 9 May 2014 14:08, Jeff Law l...@redhat.com wrote: On 05/08/14 02:07, Zhenqiang Chen wrote: Hi, The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry block. Bootstrap and no make check regression on X86-64

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

2014-05-13 Thread Zhenqiang Chen
On 9 May 2014 14:08, Jeff Law l...@redhat.com wrote: On 05/08/14 02:07, Zhenqiang Chen wrote: Hi, The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry block. Bootstrap and no make check regression on X86-64 and ARM. OK for trunk? Thanks!

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

2014-05-09 Thread Jeff Law
On 05/08/14 02:07, Zhenqiang Chen wrote: Hi, The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry block. Bootstrap and no make check regression on X86-64 and ARM. OK for trunk? Thanks! -Zhenqiang ChangeLog: 2014-05-08 Zhenqiang Chen

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

2014-05-09 Thread Zhenqiang Chen
On 9 May 2014 14:08, Jeff Law l...@redhat.com wrote: On 05/08/14 02:07, Zhenqiang Chen wrote: Hi, The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry block. Bootstrap and no make check regression on X86-64 and ARM. OK for trunk? Thanks!

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

2014-05-08 Thread Zhenqiang Chen
Hi, The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry block. Bootstrap and no make check regression on X86-64 and ARM. OK for trunk? Thanks! -Zhenqiang ChangeLog: 2014-05-08 Zhenqiang Chen zhenqiang.c...@linaro.org * function.c

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

2014-05-08 Thread Steven Bosscher
On Thu, May 8, 2014 at 10:07 AM, Zhenqiang Chen wrote: The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry block. Maybe also time to take the shrink-wrapping code out of function.c and put it in its own file? Ciao! Steven