[Mesa-dev] [PATCH 2/2] mesa: remove hack to fix up GL_ANY_SAMPLES_PASSED results

Marek Olšák maraeo at gmail.com
Fri Feb 5 10:29:14 UTC 2016


For the series:
Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Fri, Feb 5, 2016 at 7:16 AM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> Both st/mesa and i965 should return a true/false result now, and the
> only other driver implementing queries (radeon) doesn't support
> ARB_occlusion_query2 which added that pname.
>
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> ---
>  src/mesa/main/queryobj.c | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/src/mesa/main/queryobj.c b/src/mesa/main/queryobj.c
> index b86692a..7a70b59 100644
> --- a/src/mesa/main/queryobj.c
> +++ b/src/mesa/main/queryobj.c
> @@ -807,11 +807,6 @@ invalid_enum:
>        return;
>     }
>
> -   /* TODO: Have the driver be required to handle this fixup. */
> -   if (q->Target == GL_ANY_SAMPLES_PASSED ||
> -       q->Target == GL_ANY_SAMPLES_PASSED_CONSERVATIVE)
> -      value = !!value;
> -
>     switch (ptype) {
>     case GL_INT: {
>        GLint *param = (GLint *)offset;
> --
> 2.4.10
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list