On 07/13/2017 03:03 AM, Richard Sandiford wrote:
> The expand_float code:
> 
>   /* Unsigned integer, and no way to convert directly.  Convert as signed,
>      then unconditionally adjust the result.  */
> 
> and the expand_fix code:
> 
>   /* For an unsigned conversion, there is one more way to do it.
>      If we have a signed conversion, we generate code that compares
>      the real value to the largest representable positive number.  If if
>      is smaller, the conversion is done normally.  Otherwise, subtract
>      one plus the highest signed number, convert, and add it back.
> 
> are restricted to scalars, since the expansion branches on a
> comparison of the value.  This patch makes that explicit.
> 
> 2017-07-13  Richard Sandiford  <richard.sandif...@linaro.org>
>           Alan Hayward  <alan.hayw...@arm.com>
>           David Sherwood  <david.sherw...@arm.com>
> 
> gcc/
>       * optabs.c (expand_float): Explicitly check for scalars before
>       using a branching expansion.
>       (expand_fix): Likewise.
OK.
jeff

Reply via email to