Re: Simplify expmed.c:lshift_value

2013-09-09 Thread Richard Biener
On Mon, Sep 9, 2013 at 12:31 PM, Richard Sandiford wrote: > expmed.c:lshift_value has: > > val = double_int::from_uhwi (INTVAL (value)).zext (bitsize); > val = val.llshift (bitpos, HOST_BITS_PER_DOUBLE_INT); > > but its only caller has already zero-extended INTVAL (value) from BITSIZE, > so we

Simplify expmed.c:lshift_value

2013-09-09 Thread Richard Sandiford
expmed.c:lshift_value has: val = double_int::from_uhwi (INTVAL (value)).zext (bitsize); val = val.llshift (bitpos, HOST_BITS_PER_DOUBLE_INT); but its only caller has already zero-extended INTVAL (value) from BITSIZE, so we might as well pass that instead of the (value, bitsize) pair. This is