Re: [patch][combine] Fix 16-bit -> 64-bit multiply and accumulate

2011-05-19 Thread Bernd Schmidt
On 05/18/2011 11:55 AM, Andrew Stubbs wrote: > > case SIGN_EXTEND: > + op = make_compound_operation (op, SET); > + I would have expected that bit to go into the MULT case (doing it for both operands), so that the effect is limited to just making widening multiplies more obvious. Also,

Re: [patch][combine] Fix 16-bit -> 64-bit multiply and accumulate

2011-05-18 Thread Andrew Stubbs
On 03/05/11 10:07, Bernd Schmidt wrote: On 04/15/2011 12:54 PM, Andrew Stubbs wrote: Ping. Trying to unblock this... I think the point is that both examples: long long foolong (long long x, short *a, short *b) { return x + (long long)*a * (long long)*b; } and long long