<div dir="ltr">On 8 October 2013 12:17, Jordan Justen <span dir="ltr"><<a href="mailto:jljusten@gmail.com" target="_blank">jljusten@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Series Reviewed-by: Jordan Justen <<a href="mailto:jordan.l.justen@intel.com" target="_blank">jordan.l.justen@intel.com</a>><br>
<br>
I thought I might see a positive .geom with *both* per-vertex in &<br>
out. Too trivial?<br></blockquote><div><br></div><div>That's a good idea.  I'll do that as a follow-up.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><div><br>
On Wed, Oct 2, 2013 at 4:45 PM, Paul Berry <<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>> wrote:<br>
> ---<br>
>  ...gs-redeclares-pervertex-in-with-array-size.geom | 45 ++++++++++++++++++++++<br>
>  1 file changed, 45 insertions(+)<br>
>  create mode 100644 tests/spec/glsl-1.50/compiler/gs-redeclares-pervertex-in-with-array-size.geom<br>
><br>
> diff --git a/tests/spec/glsl-1.50/compiler/gs-redeclares-pervertex-in-with-array-size.geom b/tests/spec/glsl-1.50/compiler/gs-redeclares-pervertex-in-with-array-size.geom<br>
> new file mode 100644<br>
> index 0000000..f408c62<br>
> --- /dev/null<br>
> +++ b/tests/spec/glsl-1.50/compiler/gs-redeclares-pervertex-in-with-array-size.geom<br>
> @@ -0,0 +1,45 @@<br>
> +// [config]<br>
> +// expect_result: pass<br>
> +// glsl_version: 1.50<br>
> +// check_link: false<br>
> +// [end config]<br>
> +//<br>
> +// From section 7.1.1 (Compatibility Profile Built-In Language<br>
> +// Variables) of the GLSL 4.10 spec:<br>
> +//<br>
> +//     However, when a built-in interface block with an instance name<br>
> +//     is redeclared (e.g., gl_in), the instance name must be included<br>
> +//     in the redeclaration. It is an error to not include the<br>
> +//     built-in instance name or to change its name.  For example,<br>
> +//<br>
> +//     in gl_PerVertex {<br>
> +//         vec4  gl_ClipVertex;<br>
> +//         vec4  gl_FrontColor;<br>
> +//     } gl_in[];  // must be present and must be "gl_in[]"<br>
> +//<br>
> +// Although the example mentions "gl_in[]", it seems reasonable to<br>
> +// assume that specifying an array size (e.g. "gl_in[3]") is also<br>
> +// allowed (this is confirmed by tests of the NVIDIA proprietary<br>
> +// driver for Linux, version 313.18).<br>
> +//<br>
> +// Note: although this text appears in a section referring to<br>
> +// compatibility profile variables, it's clear from context that it's<br>
> +// meant to apply to any redeclaration of gl_in, whether it is done in<br>
> +// a compatibility or a core profile.<br>
> +//<br>
> +// This appears to be a clarification to the behaviour established for<br>
> +// gl_PerVertex by GLSL 1.50, therefore we test it using GLSL version<br>
> +// 1.50.<br>
> +<br>
> +#version 150<br>
> +<br>
> +layout(triangles) in;<br>
> +layout(triangle_strip, max_vertices = 3) out;<br>
> +<br>
> +in gl_PerVertex {<br>
> +  vec4 gl_Position;<br>
> +} gl_in[3];<br>
> +<br>
> +void main()<br>
> +{<br>
> +}<br>
> --<br>
> 1.8.4<br>
><br>
</div></div>> _______________________________________________<br>
> Piglit mailing list<br>
> <a href="mailto:Piglit@lists.freedesktop.org" target="_blank">Piglit@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</blockquote></div><br></div></div>