Mesa (master): glsl: Put all bitfields in ir_variable together for better packing

Ian Romanick idr at kemper.freedesktop.org
Thu Nov 3 20:36:31 UTC 2011


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

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Mon Oct 31 14:04:10 2011 -0700

glsl: Put all bitfields in ir_variable together for better packing

The diff looks weird because ir_variable::depth_layout was between the
last two bitfields in the structure.

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Acked-by: Paul Berry <stereotype441 at gmail.com>

---

 src/glsl/ir.h |   16 ++++++++--------
 1 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/glsl/ir.h b/src/glsl/ir.h
index 404d4cf..abbf455 100644
--- a/src/glsl/ir.h
+++ b/src/glsl/ir.h
@@ -356,14 +356,6 @@ public:
    /*@}*/
 
    /**
-    * \brief Layout qualifier for gl_FragDepth.
-    *
-    * This is not equal to \c ir_depth_layout_none if and only if this
-    * variable is \c gl_FragDepth and a layout qualifier is specified.
-    */
-   ir_depth_layout depth_layout;
-
-   /**
     * Was the location explicitly set in the shader?
     *
     * If the location is explicitly set in the shader, it \b cannot be changed
@@ -373,6 +365,14 @@ public:
    unsigned explicit_location:1;
 
    /**
+    * \brief Layout qualifier for gl_FragDepth.
+    *
+    * This is not equal to \c ir_depth_layout_none if and only if this
+    * variable is \c gl_FragDepth and a layout qualifier is specified.
+    */
+   ir_depth_layout depth_layout;
+
+   /**
     * Storage location of the base of this variable
     *
     * The precise meaning of this field depends on the nature of the variable.




More information about the mesa-commit mailing list