Mesa (master): r300-gallium: Use fui instead of a roll-my-own.

Corbin Simpson csimpson at kemper.freedesktop.org
Sun Feb 15 13:07:03 UTC 2009


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

Author: Corbin Simpson <MostAwesomeDude at gmail.com>
Date:   Sat Feb 14 03:23:50 2009 -0800

r300-gallium: Use fui instead of a roll-my-own.

Man, util/u_math just gets better by the day.

---

 src/gallium/drivers/r300/r300_cs.h |   16 +++-------------
 1 files changed, 3 insertions(+), 13 deletions(-)

diff --git a/src/gallium/drivers/r300/r300_cs.h b/src/gallium/drivers/r300/r300_cs.h
index 385b61a..3049702 100644
--- a/src/gallium/drivers/r300/r300_cs.h
+++ b/src/gallium/drivers/r300/r300_cs.h
@@ -23,21 +23,11 @@
 #ifndef R300_CS_H
 #define R300_CS_H
 
+#include "util/u_math.h"
+
 #include "r300_reg.h"
 #include "r300_winsys.h"
 
-/* Pack a 32-bit float into a dword. */
-static uint32_t pack_float_32(float f)
-{
-    union {
-        float f;
-        uint32_t u;
-    } u;
-
-    u.f = f;
-    return u.u;
-}
-
 /* Yes, I know macros are ugly. However, they are much prettier than the code
  * that they neatly hide away, and don't have the cost of function setup,so
  * we're going to use them. */
@@ -80,7 +70,7 @@ static uint32_t pack_float_32(float f)
 } while (0)
 
 #define OUT_CS_32F(value) do { \
-    cs_winsys->write_cs_dword(cs, pack_float_32(value)); \
+    cs_winsys->write_cs_dword(cs, fui(value)); \
     cs_count--; \
 } while (0)
 




More information about the mesa-commit mailing list