i965: Drop strict conformance fallback for GL_LINE_SMOOTH.
authorEric Anholt <eric@anholt.net>
Wed, 25 May 2011 19:54:49 +0000 (12:54 -0700)
committerEric Anholt <eric@anholt.net>
Fri, 3 Jun 2011 23:03:18 +0000 (16:03 -0700)
We do have hardware antialised lines.  If we care, we should actually
fix them to be conformant (or as close as possible) instead of using
this knob to fool testcases using swrast.

For some interesting reading on the state of GL_*_SMOOTH across
several drivers, see:
http://homepage.mac.com/arekkusu/bugs/invariance/HWAA.html

src/mesa/drivers/dri/i965/brw_draw.c

index 44d3e1b6c24579df3d2dfb990228e8c5357beeaa..b2e89bad666dfb3fb6b267d93079f6360cc394dc 100644 (file)
@@ -283,15 +283,6 @@ static GLboolean check_fallbacks( struct brw_context *brw,
    if (brw->intel.conformance_mode == 0)
       return GL_FALSE;
 
-   /* BRW hardware will do AA lines, but they are non-conformant it
-    * seems.  TBD whether we keep this fallback:
-    */
-   if (ctx->Line.SmoothFlag) {
-      for (i = 0; i < nr_prims; i++)
-        if (reduced_prim[prim[i].mode] == GL_LINES) 
-           return GL_TRUE;
-   }
-
    /* Stipple -- these fallbacks could be resolved with a little
     * bit of work?
     */