Re: PATCH: middle-end/58981: movmem/setmem use mode wider than Pmode for size

2013-11-05 Thread Richard Biener
On Tue, 5 Nov 2013, Richard Sandiford wrote: > "H.J. Lu" writes: > > This is the updated patch with md.texi change. The testcase is the same. > > Tested on x32. OK to install? > > Since I commented on the original, this looks good to me, but I can't > approve it. Approved then. Thanks, Rich

Re: PATCH: middle-end/58981: movmem/setmem use mode wider than Pmode for size

2013-11-05 Thread Richard Sandiford
"H.J. Lu" writes: > This is the updated patch with md.texi change. The testcase is the same. > Tested on x32. OK to install? Since I commented on the original, this looks good to me, but I can't approve it. Thanks, Richard

Re: PATCH: middle-end/58981: movmem/setmem use mode wider than Pmode for size

2013-11-04 Thread H.J. Lu
On Mon, Nov 4, 2013 at 3:34 AM, H.J. Lu wrote: > Y > On Mon, Nov 4, 2013 at 3:11 AM, Richard Sandiford > wrote: >> "H.J. Lu" writes: >>> emit_block_move_via_movmem and set_storage_via_setmem have >>> >>> for (mode = GET_CLASS_NARROWEST_MODE (MODE_INT); mode != VOIDmode; >>>mode = GET_M

Re: PATCH: middle-end/58981: movmem/setmem use mode wider than Pmode for size

2013-11-04 Thread H.J. Lu
Y On Mon, Nov 4, 2013 at 3:11 AM, Richard Sandiford wrote: > "H.J. Lu" writes: >> emit_block_move_via_movmem and set_storage_via_setmem have >> >> for (mode = GET_CLASS_NARROWEST_MODE (MODE_INT); mode != VOIDmode; >>mode = GET_MODE_WIDER_MODE (mode)) >> { >> enum insn_code cod

Re: PATCH: middle-end/58981: movmem/setmem use mode wider than Pmode for size

2013-11-04 Thread Richard Sandiford
"H.J. Lu" writes: > emit_block_move_via_movmem and set_storage_via_setmem have > > for (mode = GET_CLASS_NARROWEST_MODE (MODE_INT); mode != VOIDmode; >mode = GET_MODE_WIDER_MODE (mode)) > { > enum insn_code code = direct_optab_handler (movmem_optab, mode); > > if (code !=

PATCH: middle-end/58981: movmem/setmem use mode wider than Pmode for size

2013-11-04 Thread H.J. Lu
emit_block_move_via_movmem and set_storage_via_setmem have for (mode = GET_CLASS_NARROWEST_MODE (MODE_INT); mode != VOIDmode; mode = GET_MODE_WIDER_MODE (mode)) { enum insn_code code = direct_optab_handler (movmem_optab, mode); if (code != CODE_FOR_nothing /* We