The comment said they deserved to be in emit_depthbuffer, and at this point they were all there already. --- src/mesa/drivers/dri/i965/brw_vtbl.c | 15 --------------- 1 files changed, 0 insertions(+), 15 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vtbl.c b/src/mesa/drivers/dri/i965/brw_vtbl.c index c0a0b52..db0fe7e 100644 --- a/src/mesa/drivers/dri/i965/brw_vtbl.c +++ b/src/mesa/drivers/dri/i965/brw_vtbl.c @@ -95,7 +95,6 @@ brw_update_draw_buffer(struct intel_context *intel) { struct gl_context *ctx = &intel->ctx; struct gl_framebuffer *fb = ctx->DrawBuffer; - struct intel_renderbuffer *irbStencil = NULL; bool fb_has_hiz = intel_framebuffer_has_hiz(fb); if (!fb) { @@ -103,8 +102,6 @@ brw_update_draw_buffer(struct intel_context *intel) return; } - irbStencil = intel_get_renderbuffer(fb, BUFFER_STENCIL); - /* Do this here, not core Mesa, since this function is called from * many places within the driver. */ @@ -123,18 +120,6 @@ brw_update_draw_buffer(struct intel_context *intel) return; } - /* Check some stencil invariants. These should probably be in - * emit_depthbuffer(). - */ - if (irbStencil && irbStencil->mt) { - if (!intel->has_separate_stencil) - assert(irbStencil->Base.Format == MESA_FORMAT_S8_Z24); - if (fb_has_hiz || intel->must_use_separate_stencil) - assert(irbStencil->Base.Format == MESA_FORMAT_S8); - if (irbStencil->Base.Format == MESA_FORMAT_S8) - assert(intel->has_separate_stencil); - } - /* Mesa's Stencil._Enabled field is updated when * _NEW_BUFFERS | _NEW_STENCIL, but i965 code assumes that the value * only changes with _NEW_STENCIL (which seems sensible). So flag it -- 1.7.7.3 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev