commit 7d4afedb0b72d630c5ad962b594a488a4aebdc1e
author Alexis Hetu <sugoi@google.com> Wed Jun 01 13:21:03 2022 -0400
committer Alexis Hétu <sugoi@google.com> Thu Jun 02 13:37:56 2022 +0000
tree d0cb28ad665c97e7cfe85d7323991994401ccdd9
parent 28695fc5450218478c27d65e416360396c1c1046

Write stencil early when using earlyFragmentTests

The writeStencil() operation should happen early when using
earlyFragmentTests, so this CL allows this operation to happen
right after the early depth test.

Tests: dEQP-VK.fragment_operations.early_fragment.discard_early_fragment_tests_stencil
Bug: b/200826740
Change-Id: Ibfead3d9731c052b954c9d448de8dea5bb108316
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/66151
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Tested-by: Alexis Hétu <sugoi@google.com>
Presubmit-Ready: Alexis Hétu <sugoi@google.com>

src/Pipeline/PixelRoutine.cpp

diff --git a/src/Pipeline/PixelRoutine.cpp b/src/Pipeline/PixelRoutine.cpp
index 7fb54d4..7ca20f5 100644
--- a/src/Pipeline/PixelRoutine.cpp
+++ b/src/Pipeline/PixelRoutine.cpp
@@ -129,6 +129,8 @@
 				depthPass = depthPass || depthTest(zBuffer, q, x, z[q], sMask[q], zMask[q], cMask[q]);
 				depthBoundsTest(zBuffer, q, x, zMask[q], cMask[q]);
 			}
+
+			writeStencil(sBuffer, x, sMask, zMask, cMask, samples);
 		}
 
 		If(depthPass || !earlyFragmentTests)
@@ -318,7 +320,10 @@
 			}
 		}
 
-		writeStencil(sBuffer, x, sMask, zMask, cMask, samples);
+		if(!earlyFragmentTests)
+		{
+			writeStencil(sBuffer, x, sMask, zMask, cMask, samples);
+		}
 	}
 }