i965: Fix scissor-offscreen on gen6 like we did pre-gen6.
authorEric Anholt <eric@anholt.net>
Mon, 18 Oct 2010 20:11:29 +0000 (13:11 -0700)
committerEric Anholt <eric@anholt.net>
Mon, 18 Oct 2010 20:11:29 +0000 (13:11 -0700)
src/mesa/drivers/dri/i965/gen6_scissor_state.c

index 5684c2e44c8747c458e8d80157d5110c55292709..b57126c7938e054acf5d72459c7cdb332fb9470a 100644 (file)
@@ -46,7 +46,19 @@ prepare_scissor_state(struct brw_context *brw)
     * Note that the hardware's coordinates are inclusive, while Mesa's min is
     * inclusive but max is exclusive.
     */
-   if (render_to_fbo) {
+   if (ctx->DrawBuffer->_Xmin == ctx->DrawBuffer->_Xmax ||
+       ctx->DrawBuffer->_Ymin == ctx->DrawBuffer->_Ymax) {
+      /* If the scissor was out of bounds and got clamped to 0
+       * width/height at the bounds, the subtraction of 1 from
+       * maximums could produce a negative number and thus not clip
+       * anything.  Instead, just provide a min > max scissor inside
+       * the bounds, which produces the expected no rendering.
+       */
+      scissor.xmin = 1;
+      scissor.xmax = 0;
+      scissor.ymin = 1;
+      scissor.ymax = 0;
+   } else if (render_to_fbo) {
       /* texmemory: Y=0=bottom */
       scissor.xmin = ctx->DrawBuffer->_Xmin;
       scissor.xmax = ctx->DrawBuffer->_Xmax - 1;