Add support for OpFMod

Bug: b/126873455 Test: dEQP-VK.glsl.operator.common_functions.mod.* Change-Id: I7f261e7bd4dda72b18594cd7e2ee7044bc60466b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/27369Tested-by: Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Add support for OpFMod
0e4d6ffb · Chris Forbes · 10fd6247 · 0e4d6ffb
Commit 0e4d6ffb authored Mar 15, 2019 by Chris Forbes
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

SpirvShader.cpp src/Pipeline/SpirvShader.cpp +6 -0

No files found.
--- a/src/Pipeline/SpirvShader.cpp
+++ b/src/Pipeline/SpirvShader.cpp
@@ -329,6 +329,7 @@ namespace sw
 			case spv::OpFSub:
 			case spv::OpFMul:
 			case spv::OpFDiv:
+			case spv::OpFMod:
 			case spv::OpFOrdEqual:
 			case spv::OpFUnordEqual:
 			case spv::OpFOrdNotEqual:
@@ -1139,6 +1140,7 @@ namespace sw
 		case spv::OpFSub:
 		case spv::OpFMul:
 		case spv::OpFDiv:
+		case spv::OpFMod:
 		case spv::OpFOrdEqual:
 		case spv::OpFUnordEqual:
 		case spv::OpFOrdNotEqual:
@@ -1764,6 +1766,10 @@ namespace sw
 			case spv::OpFDiv:
 				dst.emplace(i, lhs.Float(i) / rhs.Float(i));
 				break;
+			case spv::OpFMod:
+				// TODO(b/126873455): inaccurate for values greater than 2^24
+				dst.emplace(i, lhs.Float(i) - rhs.Float(i) * Floor(lhs.Float(i) / rhs.Float(i)));
+				break;
 			case spv::OpFOrdEqual:
 				dst.emplace(i, CmpEQ(lhs.Float(i), rhs.Float(i)));
 				break;