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