tree 4d40df579316a360f4621383bbb32722b70afd2e
parent 5d73f03b7a6ab517798fcae032e39ca15ba03458
author Sean Risser <srisser@google.com> 1560436719 -0400
committer Sean Risser <srisser@google.com> 1560456798 +0000

Fix unsigned "less than zero" comparison

From the SPIR-V spec:
  OpSMod's result is the remainder r of Operand 1 divided by Operand 2
  where if r != 0, the sign of r is the same as the sign of Operand 2.

The less than comparison here was trying to correct the cases where C's
modulo had a different sign than SPIR-V's modulo. We can solve this by
directly comparing the sign of the C modulo against Operand 2's sign.

Bug chromium:973848

Change-Id: I27c88b7aaed35db5ba4df2cc0aac6061098f32c4
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/32868
Tested-by: Sean Risser <srisser@google.com>
Presubmit-Ready: Sean Risser <srisser@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>