Mesa (master): i965: Update comment about clipper constants.

Kenneth Graunke kwg at kemper.freedesktop.org
Thu Nov 1 21:35:24 UTC 2012


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

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Fri Oct 26 11:01:57 2012 -0700

i965: Update comment about clipper constants.

The old VS backend doesn't exist, but I believe these still need to be
delivered to the clipper thread.

Reviewed-by: Eric Anholt <eric at anholt.net>

---

 src/mesa/drivers/dri/i965/brw_curbe.c |   10 +---------
 1 files changed, 1 insertions(+), 9 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_curbe.c b/src/mesa/drivers/dri/i965/brw_curbe.c
index 8f196f4..b55453a 100644
--- a/src/mesa/drivers/dri/i965/brw_curbe.c
+++ b/src/mesa/drivers/dri/i965/brw_curbe.c
@@ -207,15 +207,7 @@ brw_upload_constant_buffer(struct brw_context *brw)
       }
    }
 
-
-   /* When using the old VS backend, the clipplanes are actually delivered to
-    * both CLIP and VS units.  VS uses them to calculate the outcode bitmasks.
-    *
-    * When using the new VS backend, it is responsible for setting up its own
-    * clipplane constants if it needs them.  This results in a slight waste of
-    * of curbe space, but the advantage is that the new VS backend can use its
-    * general-purpose uniform layout code to store the clipplanes.
-    */
+   /* clipper constants */
    if (brw->curbe.clip_size) {
       GLuint offset = brw->curbe.clip_start * 16;
       GLuint j;




More information about the mesa-commit mailing list