i965: Just assert when we flagged a compile error in the FS for now.
authorEric Anholt <eric@anholt.net>
Fri, 3 Sep 2010 21:29:57 +0000 (14:29 -0700)
committerEric Anholt <eric@anholt.net>
Tue, 7 Sep 2010 17:34:09 +0000 (10:34 -0700)
Dumping back to potentially 16-wide dispatch doesn't really work out
at the moment, and hopefully I'll just be able to resolve all the
failures so we never have to do this at all.

src/mesa/drivers/dri/i965/brw_fs.cpp

index 5b2b0e6ec49d9018dd65c64ef32ec9ccd13286a5..f0af8740d5f2b3c785c34342c2fafee43d04cdbb 100644 (file)
@@ -1871,9 +1871,6 @@ brw_wm_fs_emit(struct brw_context *brw, struct brw_wm_compile *c)
         ir->accept(&v);
       }
 
-      if (v.fail)
-        return GL_FALSE;
-
       v.emit_fb_writes();
       v.assign_curb_setup();
       v.assign_urb_setup();
@@ -1882,6 +1879,11 @@ brw_wm_fs_emit(struct brw_context *brw, struct brw_wm_compile *c)
 
    v.generate_code();
 
+   assert(!v.fail); /* FINISHME: Cleanly fail, tested at link time, etc. */
+
+   if (v.fail)
+      return GL_FALSE;
+
    if (INTEL_DEBUG & DEBUG_WM) {
       const char *last_annotation_string = NULL;
       ir_instruction *last_annotation_ir = NULL;