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

[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  <zhenqiang.c...@linaro.org>

         * regcprop.c (skip_debug_insn_p): New decl.
         (replace_oldest_value_reg): Check skip_debug_insn_p.
         (copyprop_hardreg_forward_bb_without_debug_insn.): New function.
         * shrink-wrap.c (prepare_shrink_wrap):
         Call copyprop_hardreg_forward_bb_without_debug_insn.
         * function.h (copyprop_hardreg_forward_bb_without_debug_insn):
         New prototype.

testsuite/ChangeLog:
2014-05-13  Zhenqiang Chen  <zhenqiang.c...@linaro.org>

         * shrink-wrap-loop.c: New test case.
Basically OK.

Can you create regcprop.h and put the prototype for 
copyprop_hardreg_forward_bb_without_debug_insn in there?

Then include regcprop.h in shrink-wrap.c.

With that change, approved.  Please post the final version for archival 
purposes.

jeff