Mesa (master): i965/fs: Reorder fs_reg for better packing.

Matt Turner mattst88 at kemper.freedesktop.org
Sat Feb 22 06:50:00 UTC 2014


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

Author: Matt Turner <mattst88 at gmail.com>
Date:   Wed Feb 19 18:53:00 2014 -0800

i965/fs: Reorder fs_reg for better packing.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/mesa/drivers/dri/i965/brw_fs.h |   30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h
index 5d1908a..c91a777 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.h
+++ b/src/mesa/drivers/dri/i965/brw_fs.h
@@ -85,9 +85,13 @@ public:
    bool is_contiguous() const;
 
    fs_reg &apply_stride(unsigned stride);
+   /** Smear a channel of the reg to all channels. */
+   fs_reg &set_smear(unsigned subreg);
 
    /** Register file: GRF, MRF, IMM. */
    enum register_file file;
+   /** Register type.  BRW_REGISTER_TYPE_* */
+   enum brw_reg_type type;
    /**
     * Register number.  For MRF, it's the hardware register.  For
     * GRF, it's a virtual register number until register allocation
@@ -101,14 +105,11 @@ public:
     * For uniforms, this is in units of 1 float.
     */
    int reg_offset;
-   /** Register type.  BRW_REGISTER_TYPE_* */
-   enum brw_reg_type type;
-   bool negate;
-   bool abs;
-   struct brw_reg fixed_hw_reg;
-
-   /** Smear a channel of the reg to all channels. */
-   fs_reg &set_smear(unsigned subreg);
+   /**
+    * Offset in bytes from the start of the register.  Values up to a
+    * backend_reg::reg_offset unit are valid.
+    */
+   int subreg_offset;
 
    /** Value for file == IMM */
    union {
@@ -117,16 +118,15 @@ public:
       float f;
    } imm;
 
-   /**
-    * Offset in bytes from the start of the register.  Values up to a
-    * backend_reg::reg_offset unit are valid.
-    */
-   int subreg_offset;
+   struct brw_reg fixed_hw_reg;
+
+   fs_reg *reladdr;
+
+   bool negate;
+   bool abs;
 
    /** Register region horizontal stride */
    uint8_t stride;
-
-   fs_reg *reladdr;
 };
 
 static inline fs_reg




More information about the mesa-commit mailing list