mesa: Drop incorrect A4B4G4R4 _mesa_format_matches_format_and_type() cases.
authorEric Anholt <eric@anholt.net>
Thu, 1 Feb 2018 19:12:47 +0000 (11:12 -0800)
committerEric Anholt <eric@anholt.net>
Sat, 3 Feb 2018 00:27:49 +0000 (16:27 -0800)
swapBytes operates on bytes, not 4-bit channels, so you can't just take
non-swapBytes cases and flip the REV flag.

Avoids piglit texture-packed-formats regressions when enabling the
ABGR4444 format.

Fixes: c5a5c9a7db89 ("mesa/formats: add new mesa formats and their pack/unpack functions.")
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
src/mesa/main/formats.c

index c1d8641116ec65117ae08a09054b3c3386cc1375..8d32757a8795aa1a7c732a8d6c444a7e336b9541 100644 (file)
@@ -1568,15 +1568,9 @@ _mesa_format_matches_format_and_type(mesa_format mesa_format,
       if (format == GL_RGBA && type == GL_UNSIGNED_SHORT_4_4_4_4 && !swapBytes)
          return GL_TRUE;
 
-      if (format == GL_RGBA && type == GL_UNSIGNED_SHORT_4_4_4_4_REV && swapBytes)
-         return GL_TRUE;
-
       if (format == GL_ABGR_EXT && type == GL_UNSIGNED_SHORT_4_4_4_4_REV && !swapBytes)
          return GL_TRUE;
 
-      if (format == GL_ABGR_EXT && type == GL_UNSIGNED_SHORT_4_4_4_4 && swapBytes)
-         return GL_TRUE;
-
       return GL_FALSE;
 
    case MESA_FORMAT_R4G4B4A4_UNORM: