Please ignore this one, I have just noticed that many similar tests are
auto-generated.
On 05/03/2017 03:33 PM, Samuel Pitoiset wrote:
func-param-restrict-qualifier.frag currently fails with Mesa
because the GLSL 4.50 has been relaxed the restrict qualifier.
Signed-off-by: Samuel Pitoiset <[email protected]>
---
.../compiler/func-param-additional-qualifiers.frag | 29 ++++++++++++++++++++++
.../compiler/func-param-lack-qualifiers.frag | 28 +++++++++++++++++++++
.../compiler/func-param-restrict-qualifier.frag | 23 +++++++++++++++++
3 files changed, 80 insertions(+)
create mode 100644
tests/spec/arb_shader_image_load_store/compiler/func-param-additional-qualifiers.frag
create mode 100644
tests/spec/arb_shader_image_load_store/compiler/func-param-lack-qualifiers.frag
create mode 100644
tests/spec/arb_shader_image_load_store/compiler/func-param-restrict-qualifier.frag
diff --git
a/tests/spec/arb_shader_image_load_store/compiler/func-param-additional-qualifiers.frag
b/tests/spec/arb_shader_image_load_store/compiler/func-param-additional-qualifiers.frag
new file mode 100644
index 000000000..4d6b881d3
--- /dev/null
+++
b/tests/spec/arb_shader_image_load_store/compiler/func-param-additional-qualifiers.frag
@@ -0,0 +1,29 @@
+// [config]
+// expect_result: pass
+// glsl_version: 3.30
+// require_extensions: GL_ARB_shader_image_load_store
+// [end config]
+
+#version 330
+#extension GL_ARB_shader_image_load_store: enable
+
+// From Section 4.10 (Memory Qualifiers) of the GLSL 4.50 spec:
+//
+// "It is legal to have any additional memory qualifiers on a formal
+// parameter..."
+
+layout(rgba32f) uniform image2D img1;
+layout(rgba32f) uniform image2D img2;
+layout(rgba32f) uniform image2D img3;
+writeonly uniform image2D img4;
+
+void f1(restrict coherent image2D i) {}
+void f2(volatile writeonly image2D i) {}
+
+void main()
+{
+ f1(img1);
+ f1(img2);
+ f1(img3);
+ f2(img4);
+}
diff --git
a/tests/spec/arb_shader_image_load_store/compiler/func-param-lack-qualifiers.frag
b/tests/spec/arb_shader_image_load_store/compiler/func-param-lack-qualifiers.frag
new file mode 100644
index 000000000..63e185d91
--- /dev/null
+++
b/tests/spec/arb_shader_image_load_store/compiler/func-param-lack-qualifiers.frag
@@ -0,0 +1,28 @@
+// [config]
+// expect_result: fail
+// glsl_version: 3.30
+// require_extensions: GL_ARB_shader_image_load_store
+// [end config]
+
+#version 330
+#extension GL_ARB_shader_image_load_store: enable
+
+// From Section 4.10 (Memory Qualifiers) of the GLSL 4.50 spec:
+//
+// "Variables qualified with coherent, volatile, readonly, or writeonly may
+// not be passed to functions whose formal parameters lack such qualifiers."
+
+layout(rgba32f) coherent uniform image2D img1;
+layout(rgba32f) volatile uniform image2D img2;
+layout(rgba32f) readonly uniform image2D img3;
+writeonly uniform image2D img4;
+
+void f(image2D i) {}
+
+void main()
+{
+ f(img1);
+ f(img2);
+ f(img3);
+ f(img4);
+}
diff --git
a/tests/spec/arb_shader_image_load_store/compiler/func-param-restrict-qualifier.frag
b/tests/spec/arb_shader_image_load_store/compiler/func-param-restrict-qualifier.frag
new file mode 100644
index 000000000..76f92b974
--- /dev/null
+++
b/tests/spec/arb_shader_image_load_store/compiler/func-param-restrict-qualifier.frag
@@ -0,0 +1,23 @@
+// [config]
+// expect_result: pass
+// glsl_version: 3.30
+// require_extensions: GL_ARB_shader_image_load_store
+// [end config]
+
+#version 330
+#extension GL_ARB_shader_image_load_store: enable
+
+// From Section 4.10 (Memory Qualifiers) of the GLSL 4.50 spec:
+//
+// "It is legal to have any additional memory qualifiers on a formal parameter,
+// but only restrict can be taken away from a calling argument, by a formal
+// parameter that lacks the restrict qualifier."
+
+layout(rgba32f) restrict uniform image2D img;
+
+void f(image2D i) {}
+
+void main()
+{
+ f(img);
+}
_______________________________________________
Piglit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/piglit
