Re: [PATCH, 1/2] shrink wrap a function with a single loop: copy propagation

2014-05-14 Thread Jeff Law
On 05/13/14 04:04, Zhenqiang Chen wrote: After reading the code in regcprop.c, I think I should reuse the copyprop_hardreg_forward_1. So rewrite the patch, which is much simple and should handle HAVE_cc0. But not sure we'd handle DEBUG_INSN or not. 2014-05-13 Zhenqiang Chen * regcpr

Re: [PATCH, 1/2] shrink wrap a function with a single loop: copy propagation

2014-05-13 Thread Jeff Law
On 05/09/14 01:30, Zhenqiang Chen wrote: So why restrict this to just cases where we have to propagate into a COMPARE at the end of a block? So in your example, assume the first block looks like prepare_shrink_wrap will move_insn_for_shrink_wrap in BB_INSNS_REVERSE order. Current prepare_shr

Re: [PATCH, 1/2] shrink wrap a function with a single loop: copy propagation

2014-05-13 Thread Zhenqiang Chen
After reading the code in regcprop.c, I think I should reuse the copyprop_hardreg_forward_1. So rewrite the patch, which is much simple and should handle HAVE_cc0. But not sure we'd handle DEBUG_INSN or not. 2014-05-13 Zhenqiang Chen * regcprop.c (skip_debug_insn_p): New decl.

Re: [PATCH, 1/2] shrink wrap a function with a single loop: copy propagation

2014-05-09 Thread Zhenqiang Chen
On 9 May 2014 14:00, Jeff Law wrote: > On 05/08/14 02:06, Zhenqiang Chen wrote: >> >> Hi, >> >> Similar issue was discussed in thread >> http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01145.html. The patches >> are close to Jeff's suggestion: "sink just the moves out of the >> incoming argument regi

Re: [PATCH, 1/2] shrink wrap a function with a single loop: copy propagation

2014-05-08 Thread Jeff Law
On 05/08/14 02:06, Zhenqiang Chen wrote: Hi, Similar issue was discussed in thread http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01145.html. The patches are close to Jeff's suggestion: "sink just the moves out of the incoming argument registers". The patch and following one try to shrink wrap a