[Mesa-dev] [PATCH 5/5] i965: Remove some stale comments about the brw_constant_buffer atom.

Eric Anholt eric at anholt.net
Mon Feb 4 19:31:53 PST 2013


These have been wrong since f428255bde93a452a7cdd48fba21839c99beb6cb
back in 2009!
---
 src/mesa/drivers/dri/i965/brw_curbe.c        |    6 ------
 src/mesa/drivers/dri/i965/brw_state_upload.c |    6 ------
 2 files changed, 12 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_curbe.c b/src/mesa/drivers/dri/i965/brw_curbe.c
index b55453a..b332f19 100644
--- a/src/mesa/drivers/dri/i965/brw_curbe.c
+++ b/src/mesa/drivers/dri/i965/brw_curbe.c
@@ -326,12 +326,6 @@ emit:
    ADVANCE_BATCH();
 }
 
-/* This tracked state is unique in that the state it monitors varies
- * dynamically depending on the parameters tracked by the fragment and
- * vertex programs.  This is the template used as a starting point,
- * each context will maintain a copy of this internally and update as
- * required.
- */
 const struct brw_tracked_state brw_constant_buffer = {
    .dirty = {
       .mesa = _NEW_PROGRAM_CONSTANTS,
diff --git a/src/mesa/drivers/dri/i965/brw_state_upload.c b/src/mesa/drivers/dri/i965/brw_state_upload.c
index 5b3930a..1e2a4c3 100644
--- a/src/mesa/drivers/dri/i965/brw_state_upload.c
+++ b/src/mesa/drivers/dri/i965/brw_state_upload.c
@@ -36,12 +36,6 @@
 #include "intel_batchbuffer.h"
 #include "intel_buffers.h"
 
-/* This is used to initialize brw->state.atoms[].  We could use this
- * list directly except for a single atom, brw_constant_buffer, which
- * has a .dirty value which changes according to the parameters of the
- * current fragment and vertex programs, and so cannot be a static
- * value.
- */
 static const struct brw_tracked_state *gen4_atoms[] =
 {
    &brw_wm_input_sizes,
-- 
1.7.10.4



More information about the mesa-dev mailing list