[Piglit] [PATCH 1/4] arb_shader_image_size/builtin: Fix the CubeMap image test

Francisco Jerez currojerez at riseup.net
Thu Aug 27 07:53:45 PDT 2015


Martin Peres <martin.peres at linux.intel.com> writes:

> Signed-off-by: Martin Peres <martin.peres at linux.intel.com>

Reviewed-by: Francisco Jerez <currojerez at riseup.net>

> ---
>  tests/spec/arb_shader_image_size/builtin.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/tests/spec/arb_shader_image_size/builtin.c b/tests/spec/arb_shader_image_size/builtin.c
> index cb427df..48a1d1e 100755
> --- a/tests/spec/arb_shader_image_size/builtin.c
> +++ b/tests/spec/arb_shader_image_size/builtin.c
> @@ -104,6 +104,15 @@ check(const struct grid_info grid, struct image_info img_src)
>  		 */
>  		ret &= check_pixels(img, pixels, img_src.size.x, img_src.size.y,
>  			     img_src.size.z / 6.0, img_src.size.w);
> +	} else if (img_src.target->target == GL_TEXTURE_CUBE_MAP) {
> +		/* Unlike image-load-store's size (exported by the framework
> +		 * used for this test), image-size reports only the size of one
> +		 * face, not including the number of faces. From the
> +		 * ARB_shader_image_size extension:
> +		 * "Cube images return the dimensions of one face."
> +		 */
> +		ret &= check_pixels(img, pixels, img_src.size.x, img_src.size.y,
> +			     1.0, 1.0);
>  	} else if (image_target_samples(img_src.target) > 1) {
>  		/* Unlike image-load-store's size (exported by the framework
>  		 * used for this test), image-size does not report the sample
> -- 
> 2.5.0
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150827/983c9eb6/attachment.sig>


More information about the Piglit mailing list