s3tc-errors: Fix GCC compiler warnings.

The params argument in glGetTexLevelParameteriv is of type GLint*.

This patch fixes these GCC warnings introduced with commit
4f9d98dbaea90c11a930fd774382ff364892ce3f.

s3tc-errors.c:156:6: warning: passing argument 4 of ‘piglit_dispatch_glGetTexLevelParameteriv’ from incompatible pointer type [enabled by default]
      &is_compressed);
      ^
s3tc-errors.c:156:6: note: expected ‘GLint *’ but argument is of type ‘_Bool *’
s3tc-errors.c:158:6: warning: pointer targets in passing argument 4 of ‘piglit_dispatch_glGetTexLevelParameteriv’ differ in signedness [-Wpointer-sign]
      &format);
      ^
s3tc-errors.c:158:6: note: expected ‘GLint *’ but argument is of type ‘GLenum *’
s3tc-errors.c:161:6: warning: pointer targets in passing argument 4 of ‘piglit_dispatch_glGetTexLevelParameteriv’ differ in signedness [-Wpointer-sign]
      &compressed_size);
      ^
s3tc-errors.c:161:6: note: expected ‘GLint *’ but argument is of type ‘GLuint *’

Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Daniel Kurtz <djkurtz@chromium.org>
util: add helpers for r11g11b10f

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
arb_vertex_type_10f_10f_11f_rev: add test for api-errors

V2: - Test for normalized = GL_TRUE as well
    - Don't require ARB_vertex_type_2_10_10_10_rev; if this is exposed
      without it, then the implementation is broken and we want
      failures, not skips.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Eric Anholt <eric@anholt.net>
arb_vertex_type_10f_10f_11f_rev: add test for draw-vertices

V2: - Rename `verts` to `colors`
    - Fix description of the test
    - Fix required extensions -- add ARB_explicit_attrib_location
                              -- remove ARB_vertex_type_2_10_10_10_rev
    - move packing code into util
    - probe alpha channel as well
    - draw quads, fill entire window

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
max-texture-size: test glTexSubImage2D() for GL_TEXTURE_RECTANGLE too

All the other texture targets test glTexSubImage(), so do it for
this target too.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
max-texture-size: fix size reported in error message

If texInitData() failed, it was with maxSide/2, not maxSide.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
packed_depth_stencil: fix comments
packed_depth_stencil: add a glGetTexImage test

Checks a Mesa bug in glGetTexImage(GL_DEPTH_STENCIL/GL_UNSIGNED_INT_24_8)

v2: fix the extensions checks

Reviewed-by: Eric Anholt <eric@anholt.net>
GL 3.2: Use standard internalformat when testing FramebufferTexture

GL_RGB32F is available only when texture_buffer_object_rgb32 is supported.
As the test covers different topic simply replace the format with one
that is supported by both core and compat profile and does not require
any extensions.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewd-by: Fabian Bieler <fabianbieler@fastmail.fm>
arb_clear_buffer_object: use rgb32 format when ARB_texture_buffer_object_rgb32 is available

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewd-by: Fabian Bieler <fabianbieler@fastmail.fm>
layered-rendering/blit: use color other than the default red

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
layered-rendering/blit: adding missing whitespace

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
layered-rendering/blit: remove redundant constant (texelsPerLayer)

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
layered-rendering/blit: remove redundant constant (floatPerLayer)

Also moves the allocation for the generated pixels to the
generator.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
layered-rendering/blit: specify the final fb grid using tex dimensions

This made it as least for me easier to read the individual functions
in isolation without the need of jumping back and forth checking
the definitions of "half dimensions" which in turn got divided by two
from time to time.

The sizes of the textures couldn't be changed anyway without altering
these window dimensions based arguments correspondingly.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
layered-rendering/blit: declare compile time constants as static

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
layered-rendering/blit: remove redundant tex parameters settings

The values are the defaults.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
layered-rendering/blit: remove redundant constants

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
glsl-1.50: Add compiler tests to verify gl_FragCoord redeclarations

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
glsl-1.50: Add shader tests to verify gl_FragCoord redeclarations

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>