[Mesa-dev] [PATCH 11/15] llvmpipe: remove unused subpixel_snap() and fixed_to_float()

[Samuel Pitoiset]

You are right, my fault.

On 04/11/2017 08:31 PM, Roland Scheidegger wrote:
> subpixel_snap is very much used (fixed_to_float, not so much), so you
> can't get rid of it.
> If you want you can place it into a #if !defined(PIPE_ARCH_SSE) clause
> though...
> 
> Roland
> 
> Am 11.04.2017 um 19:58 schrieb Samuel Pitoiset:
>> Fixes the following Clang warnings.
>>
>> lp_setup_tri.c:55:1: warning: unused function 'subpixel_snap' [-Wunused-function]
>> subpixel_snap(float a)
>> ^
>> lp_setup_tri.c:61:1: warning: unused function 'fixed_to_float' [-Wunused-function]
>> fixed_to_float(int a)
>> ^
>>
>> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
>> ---
>>   src/gallium/drivers/llvmpipe/lp_setup_tri.c | 12 ------------
>>   1 file changed, 12 deletions(-)
>>
>> diff --git a/src/gallium/drivers/llvmpipe/lp_setup_tri.c b/src/gallium/drivers/llvmpipe/lp_setup_tri.c
>> index 98243a12de..cc1508ccc2 100644
>> --- a/src/gallium/drivers/llvmpipe/lp_setup_tri.c
>> +++ b/src/gallium/drivers/llvmpipe/lp_setup_tri.c
>> @@ -51,18 +51,6 @@
>>   #include "util/u_pwr8.h"
>>   #endif
>>   
>> -static inline int
>> -subpixel_snap(float a)
>> -{
>> -   return util_iround(FIXED_ONE * a);
>> -}
>> -
>> -static inline float
>> -fixed_to_float(int a)
>> -{
>> -   return a * (1.0f / FIXED_ONE);
>> -}
>> -
>>   
>>   /* Position and area in fixed point coordinates */
>>   struct fixed_position {
>>
>