Do you have commit access?
Marek
On Thu, Feb 15, 2018 at 12:40 AM, gurchetansi...@chromium.org
<gurchetansi...@chromium.org> wrote:
> From: Gurchetan Singh <gurchetansi...@chromium.org>
>
> The ARB_viewport_array spec says:
>
> "Dependencies
> OpenGL 1.0 is required.
>
> OpenGL 3.2 or the EXT_geometry_shader4 or ARB_geometry_shader4 extensions
> are required.
>
> This extension is written against the OpenGL 3.2 (Compatibility)
> Specification."
>
> As such, we should ignore it for GLES2 contexts.
>
> Fixes:
> dEQP-GLES2.functional.state_query.integers.viewport_getinteger
> dEQP-GLES2.functional.state_query.integers.viewport_getfloat
>
> on llvmpipe and virgl.
>
> v2: Use _mesa_has_* (Ilia)
> ---
> src/mesa/main/viewport.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/main/viewport.c b/src/mesa/main/viewport.c
> index fc384909e6..398cc63685 100644
> --- a/src/mesa/main/viewport.c
> +++ b/src/mesa/main/viewport.c
> @@ -51,9 +51,8 @@ clamp_viewport(struct gl_context *ctx, GLfloat *x, GLfloat
> *y,
> * determined by calling GetFloatv with the symbolic constant
> * VIEWPORT_BOUNDS_RANGE (see section 6.1)."
> */
> - if (ctx->Extensions.ARB_viewport_array ||
> - (ctx->Extensions.OES_viewport_array &&
> - _mesa_is_gles31(ctx))) {
> + if (_mesa_has_ARB_viewport_array(ctx) ||
> + _mesa_has_OES_viewport_array(ctx)) {
> *x = CLAMP(*x,
> ctx->Const.ViewportBounds.Min,
> ctx->Const.ViewportBounds.Max);
> *y = CLAMP(*y,
> --
> 2.13.5
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
