i965: Update comment about clipper constants.
authorKenneth Graunke <kenneth@whitecape.org>
Fri, 26 Oct 2012 18:01:57 +0000 (11:01 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 1 Nov 2012 21:29:32 +0000 (14:29 -0700)
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@anholt.net>
src/mesa/drivers/dri/i965/brw_curbe.c

index 8f196f4830b8136993fa4f6a7d5305ebb71ef598..b55453aece4f2ecf97a0deed16bcd8233e0eaedb 100644 (file)
@@ -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;