i965/blorp/clear: Stop stomping the destination format
authorJason Ekstrand <jason.ekstrand@intel.com>
Mon, 25 Jul 2016 21:05:48 +0000 (14:05 -0700)
committerJason Ekstrand <jason.ekstrand@intel.com>
Wed, 17 Aug 2016 21:46:22 +0000 (14:46 -0700)
The blorp_surface_info_init call above should set the format for us and
stomping it later does nothing whatsoever.

Reviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
src/mesa/drivers/dri/i965/brw_blorp_clear.cpp

index 617cf4b7c6b0b99a4022c80e4bfec161cfaf563e..df491d1393b84735cceb0a827945028cd7b09285 100644 (file)
@@ -132,6 +132,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
    struct brw_blorp_params params;
    brw_blorp_params_init(&params);
 
+   /* Override the surface format according to the context's sRGB rules. */
    if (!encode_srgb && _mesa_get_format_color_encoding(format) == GL_SRGB)
       format = _mesa_get_srgb_format_linear(format);
 
@@ -220,9 +221,6 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
                                brw_blorp_to_isl_format(brw, format, true),
                                true);
 
-   /* Override the surface format according to the context's sRGB rules. */
-   params.dst.view.format = (enum isl_format)brw->render_target_format[format];
-
    const char *clear_type;
    if (is_fast_clear)
       clear_type = "fast";