Mesa (master): llvmpipe: remove unused subpixel_snap() and fixed_to_float( )

Samuel Pitoiset hakzsam at kemper.freedesktop.org
Thu Apr 13 08:06:58 UTC 2017


Module: Mesa
Branch: master
Commit: fbe2ff7740edfe5096dfbef4d02b478ab7936efd
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=fbe2ff7740edfe5096dfbef4d02b478ab7936efd

Author: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Date:   Tue Apr 11 14:30:42 2017 +0200

llvmpipe: remove unused subpixel_snap() and fixed_to_float()

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 <samuel.pitoiset at gmail.com>
Reviewed-by: Roland Scheidegger <sroland at vmware.com>

---

 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 {




More information about the mesa-commit mailing list