On Tue, Apr 2, 2013 at 7:43 AM, Wei Mi <w...@google.com> wrote:
> I attached the patch.4 based on r197308. r197308 changes shift-and
> type truncation from define_insn_and_split to define_insn.  patch.4
> changes ix86_rtx_costs for shift-and type rtx to get the correct cost
> for the result after the shift-and truncation.
>
> With the patch.1 ~ patch.4, fwprop extension could handle the
> motivational case 1.c attached by removing all the redundent "x & 63"
> operations.
>
> patch.1~patch.4 regression and bootstrap ok on
> x86_64-unknown-linux-gnu. Is it ok for trunk?

> 2013-04-01  Wei Mi  <w...@google.com>
>
>     * config/i386/i386.c (ix86_rtx_costs): Set proper rtx cost for
>     ashl<mode>3_mask, *<shift_insn><mode>3_mask and
    *<rotate_insn><mode>3_mask in i386.md.

Patch 4 is OK for mainline and also for release branches that were
changed by your previous i386.md patch.

Thanks,
Uros.

Reply via email to