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.