mesa: Drop redundant checks for sRGB before sRGB to linear conversion.
authorEric Anholt <eric@anholt.net>
Mon, 1 Jul 2019 22:38:56 +0000 (15:38 -0700)
committerEric Anholt <eric@anholt.net>
Tue, 16 Jul 2019 19:51:13 +0000 (12:51 -0700)
_mesa_get_srgb_format_linear() just returns the original format if it
wasn't sRGB.

Reviewed-by: Thomas Helland <thomashelland90@gmail.com>
Reviewed-by: Kristian H. Kristensen <hoegsberg@google.com>
src/mesa/drivers/dri/i965/brw_blorp.c
src/mesa/swrast/s_texfetch.c

index c32e3d1f9c06937c071bd8db5262657239936927..044b9d045e359a40b7bbc643acacefd0cc52885b 100644 (file)
@@ -299,10 +299,10 @@ brw_blorp_blit_miptrees(struct brw_context *brw,
        dst_level, dst_layer, dst_x0, dst_y0, dst_x1, dst_y1,
        mirror_x, mirror_y);
 
-   if (!decode_srgb && _mesa_get_format_color_encoding(src_format) == GL_SRGB)
+   if (!decode_srgb)
       src_format = _mesa_get_srgb_format_linear(src_format);
 
-   if (!encode_srgb && _mesa_get_format_color_encoding(dst_format) == GL_SRGB)
+   if (!encode_srgb)
       dst_format = _mesa_get_srgb_format_linear(dst_format);
 
    /* When doing a multisample resolve of a GL_LUMINANCE32F or GL_INTENSITY32F
@@ -1191,7 +1191,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
    uint32_t x0, x1, y0, y1;
 
    mesa_format format = irb->Base.Base.Format;
-   if (!encode_srgb && _mesa_get_format_color_encoding(format) == GL_SRGB)
+   if (!encode_srgb)
       format = _mesa_get_srgb_format_linear(format);
    enum isl_format isl_format = brw->mesa_to_isl_render_format[format];
 
index ed8df7bab64c9b73c229153e67f238af3062c14e..079df32f7cd39eadfc6a7081ee587c33f2fbc995 100644 (file)
@@ -491,10 +491,8 @@ set_fetch_functions(const struct gl_sampler_object *samp,
 
    STATIC_ASSERT(ARRAY_SIZE(texfetch_funcs) == MESA_FORMAT_COUNT);
 
-   if (samp->sRGBDecode == GL_SKIP_DECODE_EXT &&
-       _mesa_get_format_color_encoding(format) == GL_SRGB) {
+   if (samp->sRGBDecode == GL_SKIP_DECODE_EXT)
       format = _mesa_get_srgb_format_linear(format);
-   }
 
    assert(format < MESA_FORMAT_COUNT);