On Mon, Apr 4, 2011 at 9:57 PM, Steven Bosscher <stevenb....@gmail.com> wrote:
> Hi,
>
> This patch has one significant change: cprop_insn would not iterate
> after a successful copy propagation, assuming the uses found with
> find_used_regs remain the same. This is not true in two cases:
>
> * find_used_regs found multiple uses of the same register, but that
> register was replaced by try_replace_reg.
>
> * find_used_regs found regs that have been folded after substitution
> in try_replace_reg (which ultimately tries simplify_replace_rtx if all
> else fails)
>
> After this patch, cprop_insn starts over after one successful copy
> propagation replacement.
>
>
> The rest of the patch just cleans up a single-member struct reg_use {
> rtx reg }. An array of rtx is simpler, although it probably makes no
> difference for the generated code for cprop.c.
>
> Bootstrapped & tested on x86_64-unknown-linux-gnu. OK?Ok. Thanks, Richard. > Ciao! > Steven >
