commit 0f70a7fa78b484008ac440352979e8bd62e4f0cd
author Nicolas Capens <capn@google.com> Wed Jul 26 13:50:04 2017 -0400
committer Nicolas Capens <nicolascapens@google.com> Wed Jul 26 20:12:10 2017 +0000
tree f4f2334ae3e07ae4d0414ab87b65541b9b1ae117
parent 9f737d3718ea156d8a91bcef87022df3a6b7aee6

Fix sign masks of narrow vectors.

The SignMask intrinsic returns the most significant bit of each byte in
a 128-bit vector. For 'emulated' 8-byte vectors we expect the upper
half to be ignored.

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

src/Reactor/SubzeroReactor.cpp

diff --git a/src/Reactor/SubzeroReactor.cpp b/src/Reactor/SubzeroReactor.cpp
index 49163a7..31f0aaf 100644
--- a/src/Reactor/SubzeroReactor.cpp
+++ b/src/Reactor/SubzeroReactor.cpp
@@ -289,7 +289,6 @@
 			}
 		}
 
-
 		return symbolValue;
 	}
 
@@ -2865,7 +2864,7 @@
 			movmsk->addArg(x.value);
 			::basicBlock->appendInst(movmsk);
 
-			return RValue<Int>(V(result));
+			return RValue<Int>(V(result)) & 0xFF;
 		}
 	}
 
@@ -3141,7 +3140,7 @@
 			movmsk->addArg(x.value);
 			::basicBlock->appendInst(movmsk);
 
-			return RValue<Int>(V(result));
+			return RValue<Int>(V(result)) & 0xFF;
 		}
 	}