Drop checking ContainsSampleQualifier ... because this is covered by the SampleRateShading capability check. Bug: b/358602678 Test: dEQP-VK.draw.dynamic_rendering.complete_secondary_cmd_buff.implicit_sample_shading#sample_id_static_use Test: dEQP-VK.draw.dynamic_rendering.complete_secondary_cmd_buff.implicit_sample_shading#sample_position_static_use Test: dEQP-VK.draw.dynamic_rendering.partial_secondary_cmd_buff.implicit_sample_shading#sample_id_static_use Test: dEQP-VK.draw.dynamic_rendering.partial_secondary_cmd_buff.implicit_sample_shading#sample_position_static_use Test: dEQP-VK.draw.dynamic_rendering.primary_cmd_buff.implicit_sample_shading#sample_id_static_use Test: dEQP-VK.draw.dynamic_rendering.primary_cmd_buff.implicit_sample_shading#sample_position_static_use Test: dEQP-VK.draw.renderpass.implicit_sample_shading#sample_id_static_use Test: dEQP-VK.draw.renderpass.implicit_sample_shading#sample_position_static_use Change-Id: Ie62dd8faf3b117343744327808c16ab98a692e95 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/73968 Presubmit-Ready: Jason Macnak <natsu@google.com> Kokoro-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Chris Forbes <chrisforbes@google.com> Commit-Queue: Jason Macnak <natsu@google.com> Tested-by: Jason Macnak <natsu@google.com>
diff --git a/src/Pipeline/PixelRoutine.cpp b/src/Pipeline/PixelRoutine.cpp index c7058ac..aed652a 100644 --- a/src/Pipeline/PixelRoutine.cpp +++ b/src/Pipeline/PixelRoutine.cpp
@@ -36,7 +36,7 @@ if(spirvShader) { - if(spirvShader->getUsedCapabilities().InterpolationFunction) // TODO(b/194714095) + if(spirvShader->getUsedCapabilities().InterpolationFunction) // TODO(b/194714095) { return true; } @@ -45,11 +45,6 @@ { return true; } - - if(spirvShader->getAnalysis().ContainsSampleQualifier) - { - return true; - } } return false;