<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 10, 2015 at 2:22 AM, Chris Forbes <span dir="ltr"><<a href="mailto:chrisf@ijw.co.nz" target="_blank">chrisf@ijw.co.nz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Sat, Jan 10, 2015 at 7:27 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> This adds a test that uses an indirect offset into an array of 2D shadow<br>
> samplers with a textureProjGradOffset.  The primary purpse of this is to<br>
> use a lot of arguments with an indirect offset.  This hits a bug in the<br>
> current NIR code where we originally assumed a max of 4 texture arguments.<br>
> ---<br>
>  .../compiler/indirect-projGradOffset-shadow.frag       | 18 ++++++++++++++++++<br>
>  1 file changed, 18 insertions(+)<br>
>  create mode 100644 tests/spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag<br>
><br>
> diff --git a/tests/spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag b/tests/spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag<br>
> new file mode 100644<br>
> index 0000000..43bbe0b<br>
> --- /dev/null<br>
> +++ b/tests/spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag<br>
> @@ -0,0 +1,18 @@<br>
> +// [config]<br>
> +// expect_result: pass<br>
> +// glsl_version: 1.30<br>
> +// [end config]<br>
> +#version 130<br>
> +#extension GL_ARB_gpu_shader5: require<br>
> +<br>
> +uniform int i;<br>
> +uniform vec4 coord;<br>
> +uniform ivec2 offset;<br>
<br>
</div></div>Nothing uses this?<br></blockquote><div><br></div><div>right.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> +uniform sampler2DShadow s[5];<br>
> +<br>
> +void main()<br>
> +{<br>
> +  float f = textureProjGradOffset(s[i], coord, coord.xy, <a href="http://coord.zw" target="_blank">coord.zw</a>,<br>
> +                                 ivec2(43, 7));<br>
<br>
</span>The offset components should really be restricted to -8/+7.<br></blockquote><div><br></div><div>sure.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> +  gl_FragColor = vec4(0, f, 0, 1);<br>
> +}<br>
> --<br>
> 2.2.0<br>
<br>
</span>Otherwise,<br>
<br>
Reviewed-by: Chris Forbes <<a href="mailto:chrisf@ijw.co.nz">chrisf@ijw.co.nz</a>><br></blockquote><div><br></div><div>Thanks.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
><br>
> _______________________________________________<br>
> Piglit mailing list<br>
> <a href="mailto:Piglit@lists.freedesktop.org">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>