On 1/2/19 7:39 AM, Danylo Piliaiev wrote:
> Test that TCS per-vertex outputs which are used only in
> TCS stage are not converted to local variables and indeed
> share data within the patch.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104297
> Signed-off-by: Danylo Piliaiev <danylo.pilia...@globallogic.com>
> ---
> .../tcs-output-unmatched.shader_test | 65 +++++++++++++++++++
> 1 file changed, 65 insertions(+)
> create mode 100644
> tests/spec/arb_tessellation_shader/execution/tcs-output-unmatched.shader_test
>
> diff --git
> a/tests/spec/arb_tessellation_shader/execution/tcs-output-unmatched.shader_test
>
> b/tests/spec/arb_tessellation_shader/execution/tcs-output-unmatched.shader_test
> new file mode 100644
> index 000000000..de1b0143b
> --- /dev/null
> +++
> b/tests/spec/arb_tessellation_shader/execution/tcs-output-unmatched.shader_test
> @@ -0,0 +1,65 @@
> +# Test that TCS per-vertex outputs which are used only in
> +# TCS stage are not converted to local variables
> +# and indeed share data within the patch.
> +
> +[require]
> +GLSL >= 1.50
> +GL_ARB_tessellation_shader
> +GL_ARB_separate_shader_objects
> +
> +[vertex shader passthrough]
> +
> +
> +[tessellation control shader]
> +#extension GL_ARB_tessellation_shader: require
> +#extension GL_ARB_separate_shader_objects: require
> +#extension GL_ARB_shader_atomic_counters: require
This should be listed in the [require] section.
> +#extension GL_ARB_shader_atomic_counter_ops: require
Only atomicCounterIncrement is used, so GL_ARB_shader_atomic_counter_ops
is not needed.
> +layout(vertices = 3) out;
> +
> +layout(location=0) out vec4 out_unused_in_tes[];
For consistency, 'location = 0'.
> +
> +layout(binding = 0) uniform atomic_uint mismatches;
> +
> +void main() {
> + out_unused_in_tes[gl_InvocationID] = gl_in[gl_InvocationID].gl_Position;
> +
> + barrier();
> +
> + if(out_unused_in_tes[0] != gl_in[0].gl_Position ||
^ space here
> + out_unused_in_tes[1] != gl_in[1].gl_Position ||
> + out_unused_in_tes[2] != gl_in[2].gl_Position) {
> + atomicCounterIncrement(mismatches);
> + }
> +}
> +
> +
> +[tessellation evaluation shader]
> +#extension GL_ARB_tessellation_shader: require
> +layout(triangles) in;
> +
> +void main() {
> + gl_Position = vec4(0);
> +}
> +
> +
> +[fragment shader]
> +
> +void main()
> +{
> + gl_FragColor = vec4(1.0);
> +}
> +
> +[vertex data]
> +piglit_vertex/float/4
> +1 1 1 1
> +2 2 2 2
> +3 3 3 3
> +
> +[test]
> +atomic counters 1
> +
> +patch parameter vertices 3
> +draw arrays GL_PATCHES 0 3
> +
> +probe atomic counter 0 == 0
>
_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit
