From: Brian Paul <brianp@vmware.com>
Date: Thu, 13 Oct 2011 23:16:17 +0000 (-0600)
Subject: swrast: be a bit smarter in clip_span()
X-Git-Tag: mesa-8.0-rc1~1712
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=718cbe4ba95439c074144ea2ce06ebf0915a660c;p=platform%2Fupstream%2Fmesa.git

swrast: be a bit smarter in clip_span()

If no pixels pass the clip test, return false.
---

diff --git a/src/mesa/swrast/s_span.c b/src/mesa/swrast/s_span.c
index 1aa20f9..e517c9a 100644
--- a/src/mesa/swrast/s_span.c
+++ b/src/mesa/swrast/s_span.c
@@ -707,11 +707,13 @@ clip_span( struct gl_context *ctx, SWspan *span )
       const GLint n = span->end;
       GLubyte *mask = span->array->mask;
       GLint i;
+      GLuint passed = 0;
       if (span->arrayMask & SPAN_MASK) {
          /* note: using & intead of && to reduce branches */
          for (i = 0; i < n; i++) {
             mask[i] &= (x[i] >= xmin) & (x[i] < xmax)
                      & (y[i] >= ymin) & (y[i] < ymax);
+            passed += mask[i];
          }
       }
       else {
@@ -719,9 +721,10 @@ clip_span( struct gl_context *ctx, SWspan *span )
          for (i = 0; i < n; i++) {
             mask[i] = (x[i] >= xmin) & (x[i] < xmax)
                     & (y[i] >= ymin) & (y[i] < ymax);
+            passed += mask[i];
          }
       }
-      return GL_TRUE;  /* some pixels visible */
+      return passed > 0;
    }
    else {
       /* horizontal span of pixels */