Reviewed-by: Roland Scheidegger <[email protected]>

Am 11.04.2017 um 21:59 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)
> ^
> 
> v2: - do not remove subpixel_snap() (use !PIPE_ARCH_SSE instead)
> 
> Signed-off-by: Samuel Pitoiset <[email protected]>
> ---
>  src/gallium/drivers/llvmpipe/lp_setup_tri.c | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/src/gallium/drivers/llvmpipe/lp_setup_tri.c 
> b/src/gallium/drivers/llvmpipe/lp_setup_tri.c
> index 98243a12de..a7a5d05c32 100644
> --- a/src/gallium/drivers/llvmpipe/lp_setup_tri.c
> +++ b/src/gallium/drivers/llvmpipe/lp_setup_tri.c
> @@ -51,18 +51,15 @@
>  #include "util/u_pwr8.h"
>  #endif
>  
> +#if !defined(PIPE_ARCH_SSE)
> +
>  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);
> -}
> -
> +#endif
>  
>  /* Position and area in fixed point coordinates */
>  struct fixed_position {
> 

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to