commit 494e4b5f802b74a6b03773d4e74bfe0ec51a2340
author Alexis Hetu <sugoi@google.com> Tue Aug 01 16:49:17 2017 -0400
committer Alexis Hétu <sugoi@google.com> Tue Aug 01 21:08:48 2017 +0000
tree 0893867675310fd3ea4c96708dad0e593433f0d7
parent 0180577265cdccc08038b0bdba96b40c68ea675e

Fixed loading GL_RGB10_A2UI texture format

The GL_RGB10_A2UI was being unpacked as a double sized version of
GL_UNSIGNED_SHORT_5_5_5_1, which it is not. The channels are not
stored in the same order for these 2 formats.

This fixes all "rgb10_a2ui" related test in:
dEQP-GLES3.functional.texture.format.sized.*
dEQP-GLES3.functional.texture.specification.*

Change-Id: I0d398d0537a440906e9ef3aeb01fe4e782f778b6
Reviewed-on: https://swiftshader-review.googlesource.com/11168
Tested-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>

src/OpenGL/common/Image.cpp

diff --git a/src/OpenGL/common/Image.cpp b/src/OpenGL/common/Image.cpp
index d3fe20e..f75f92f 100644
--- a/src/OpenGL/common/Image.cpp
+++ b/src/OpenGL/common/Image.cpp
@@ -263,10 +263,10 @@
 		for(int x = 0; x < width; x++)
 		{
 			unsigned int rgba = source1010102U[x];
-			dest16U[4 * x + 0] = (rgba & 0x00000FFC) >> 2;
-			dest16U[4 * x + 1] = (rgba & 0x003FF000) >> 12;
-			dest16U[4 * x + 2] = (rgba & 0xFFC00000) >> 22;
-			dest16U[4 * x + 3] = (rgba & 0x00000003);
+			dest16U[4 * x + 0] = (rgba & 0x000003FF);
+			dest16U[4 * x + 1] = (rgba & 0x000FFC00) >> 10;
+			dest16U[4 * x + 2] = (rgba & 0x3FF00000) >> 20;
+			dest16U[4 * x + 3] = (rgba & 0xC0000000) >> 30;
 		}
 	}