commit 18abaf5ab304d48740a3c5007f6557291a7e4bff
author Nicolas Capens <capn@google.com> Wed Dec 20 11:07:45 2017 -0500
committer Nicolas Capens <nicolascapens@google.com> Wed Dec 20 20:58:15 2017 +0000
tree ff7c9a7514937ead63fb3bb6c4d096b97545f2d7
parent a141a0755a1a463894bbe22990c1d9286e493e0b

Fix format mapping for 10F_11F_11F and RGB9_E5.

GL_UNSIGNED_INT_10F_11F_11F_REV and GL_UNSIGNED_INT_5_9_9_9_REV were
using sw::FORMAT_A32B32G32R32F as the implementation format, which
resulted in using bad alpha values when it was rendered into.

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

src/OpenGL/common/Image.cpp

diff --git a/src/OpenGL/common/Image.cpp b/src/OpenGL/common/Image.cpp
index a56f8b7..ff957a9 100644
--- a/src/OpenGL/common/Image.cpp
+++ b/src/OpenGL/common/Image.cpp
@@ -913,8 +913,8 @@
 				return sw::FORMAT_A2B10G10R10;
 			}
 		case GL_UNSIGNED_INT_10F_11F_11F_REV:
-		case GL_UNSIGNED_INT_5_9_9_9_REV:
-			return sw::FORMAT_A32B32G32R32F;
+		case GL_UNSIGNED_INT_5_9_9_9_REV:   // 5 is the exponent field, not alpha.
+			return sw::FORMAT_X32B32G32R32F;
 		default:
 			UNREACHABLE(type);
 		}