Protect against DBZ in OpUMod also

Bug: b/127962486 Test: dEQP-VK.glsl.operator.binary_operator.mod.* Change-Id: I6aee79c99b50f936512e9d9b4b51a07006445694 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/27208Tested-by: Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Protect against DBZ in OpUMod also
3ebf583c · Chris Forbes · eb7cf3c4 · 3ebf583c
Commit 3ebf583c authored Mar 14, 2019 by Chris Forbes
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

SpirvShader.cpp src/Pipeline/SpirvShader.cpp +4 -1

No files found.
--- a/src/Pipeline/SpirvShader.cpp
+++ b/src/Pipeline/SpirvShader.cpp
@@ -1566,8 +1566,11 @@ namespace sw
 				break;
 			}
 			case spv::OpUMod:
-				dst.emplace(i, lhs.UInt(i) % rhs.UInt(i));
+			{
+				auto zeroMask = As<SIMD::UInt>(CmpEQ(rhs.Int(i), SIMD::Int(0)));
+				dst.emplace(i, lhs.UInt(i) % (rhs.UInt(i) | zeroMask));
 				break;
+			}
 			case spv::OpIEqual:
 			case spv::OpLogicalEqual:
 				dst.emplace(i, CmpEQ(lhs.Int(i), rhs.Int(i)));