commit 2d8c370e1f949b9ddf7249179fa5a6041b4a050d
author Nicolas Capens <capn@google.com> Tue Jul 25 13:56:46 2017 -0400
committer Nicolas Capens <nicolascapens@google.com> Wed Jul 26 19:28:11 2017 +0000
tree 294b4d220b3c5d6f10ac0c99683387e024aaf93a
parent 8772202b4bf74c5e8e00da32edc74e147d439f49

Support bitcasts between scalars and short vectors.

Subzero only supports 128-bit vectors currently, but we need to support
bitcasting between scalars and (emulated) short vectors of the same
size. Subzero implicitly supports it on x86 by using movd instructions,
but on ARM we have to emulate it in Reactor until support for it is
added in Subzero.

Bug swiftshader:48

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

src/Shader/SamplerCore.cpp

diff --git a/src/Shader/SamplerCore.cpp b/src/Shader/SamplerCore.cpp
index 0ee9014..9c58e70 100644
--- a/src/Shader/SamplerCore.cpp
+++ b/src/Shader/SamplerCore.cpp
@@ -1937,16 +1937,16 @@
 					{
 					case FORMAT_R8I:
 					case FORMAT_R8UI:
-					{
-						Int zero(0);
-						c.x = Unpack(As<Byte4>(c0), As<Byte4>(zero));
-						// Propagate sign bit
-						if(state.textureFormat == FORMAT_R8I)
 						{
-							c.x = (c.x << 8) >> 8;
+							Int zero(0);
+							c.x = Unpack(As<Byte4>(c0), As<Byte4>(zero));
+							// Propagate sign bit
+							if(state.textureFormat == FORMAT_R8I)
+							{
+								c.x = (c.x << 8) >> 8;
+							}
 						}
-					}
-					break;
+						break;
 					default:
 						c.x = Unpack(As<Byte4>(c0));
 						break;