Mesa (master): i965/fs: Update an ancient, wrong comment about reg_offset.

Eric Anholt anholt at kemper.freedesktop.org
Thu Oct 31 01:09:52 UTC 2013


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

Author: Eric Anholt <eric at anholt.net>
Date:   Tue Oct 29 01:06:09 2013 -0700

i965/fs: Update an ancient, wrong comment about reg_offset.

This hasn't been true since SIMD16 mode was added.

Reviewed-by: Paul Berry <stereotype441 at gmail.com>

---

 src/mesa/drivers/dri/i965/brw_fs.h |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h
index 5b78313..5f331e1 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.h
+++ b/src/mesa/drivers/dri/i965/brw_fs.h
@@ -88,9 +88,11 @@ public:
     */
    int reg;
    /**
-    * For virtual registers, this is a hardware register offset from
-    * the start of the register block (for example, a constant index
-    * in an array access).
+    * Offset from the start of the contiguous register block.
+    *
+    * For pre-register-allocation GRFs, this is in units of a float per pixel
+    * (1 hardware register for SIMD8 mode, or 2 registers for SIMD16 mode).
+    * For uniforms, this is in units of 1 float.
     */
    int reg_offset;
    /** Register type.  BRW_REGISTER_TYPE_* */




More information about the mesa-commit mailing list