Richard Biener <richard.guent...@gmail.com> writes:
> Richard Sandiford <rdsandif...@googlemail.com> wrote:
>>Richard Biener <richard.guent...@gmail.com> writes:
>>> On Fri, Nov 29, 2013 at 12:14 PM, Richard Sandiford
>>> <rdsandif...@googlemail.com> wrote:
>>>> In the fold-const.ii testcase, well over half of the mul_internal
>>calls
>>>> were for multiplication by 0 (106038 out of 169355).  This patch
>>adds
>>>> an early-out for that.
>>>>
>>>> Tested on x86_64-linux-gnu.  OK to install?
>>>
>>> Ok.  Did you check how many of the remaining are multiplies by 1?
>>
>>Turns out to be 9685, which is probably enough to justify a special
>>case.
>>
>>Tested on x86_64-linux-gnu.  OK to install?
>
> Ok.  I assume we already have a special.case for division by 1?

I don't think it's really worth having one there.  If the inputs fit
into single HWIs then the output does too, so we can just use a host
division.

Thanks,
Richard

Reply via email to