From: Roland Scheidegger <sroland@vmware.com>
Date: Thu, 3 Dec 2009 22:26:13 +0000 (+0100)
Subject: mesa: use _mesa_memcpy for COPY_4FV macro
X-Git-Tag: 062012170305~15615^2~7
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=86c8f70db10a584aa78e4d5f397ad3543fdb77d2;p=profile%2Fivi%2Fmesa.git

mesa: use _mesa_memcpy for COPY_4FV macro

Gets rid of one of the worst strict-aliasing offenders, and actually
produces faster code (at least in some cases, when compiler can use
for instance 64bit moves for memcpy).
(note _mesa_memcpy should get inlined)
---

diff --git a/src/mesa/main/macros.h b/src/mesa/main/macros.h
index 3d9a1ab..f0ea463 100644
--- a/src/mesa/main/macros.h
+++ b/src/mesa/main/macros.h
@@ -202,17 +202,12 @@ do {                                \
 #endif
 
 /**
- * Copy a 4-element float vector (avoid using FPU registers)
- * XXX Could use two 64-bit moves on 64-bit systems
+ * Copy a 4-element float vector
+ * memcpy seems to be most efficient
  */
 #define COPY_4FV( DST, SRC )                  \
 do {                                          \
-   const GLuint *_s = (const GLuint *) (SRC); \
-   GLuint *_d = (GLuint *) (DST);             \
-   _d[0] = _s[0];                             \
-   _d[1] = _s[1];                             \
-   _d[2] = _s[2];                             \
-   _d[3] = _s[3];                             \
+   _mesa_memcpy(DST, SRC, sizeof(GLfloat) * 4);       \
 } while (0)
 
 /** Copy \p SZ elements into a 4-element vector */