Hey Dave,
Dave Airlie wrote on 23.08.2016 03:28:
> From: Dave Airlie <airl...@redhat.com>
> 
> This should be all that is required for cull distances to work
> on radeonsi.

in case this is indeed enough to enable ARB_cull_distance, mind adding that to
GL3.txt and the release notes?

> Signed-off-by: Dave Airlie <airl...@redhat.com>
> ---
>  src/gallium/drivers/radeonsi/si_pipe.c  | 3 ++-
>  src/gallium/drivers/radeonsi/si_state.c | 8 ++++----
>  2 files changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/src/gallium/drivers/radeonsi/si_pipe.c 
> b/src/gallium/drivers/radeonsi/si_pipe.c
> index 62b62db..14b0b80 100644
> --- a/src/gallium/drivers/radeonsi/si_pipe.c
> +++ b/src/gallium/drivers/radeonsi/si_pipe.c
> @@ -436,7 +436,6 @@ static int si_get_param(struct pipe_screen* pscreen, enum 
> pipe_cap param)
>       case PIPE_CAP_TEXTURE_GATHER_OFFSETS:
>       case PIPE_CAP_VERTEXID_NOBASE:
>       case PIPE_CAP_QUERY_BUFFER_OBJECT:
> -     case PIPE_CAP_CULL_DISTANCE:
>       case PIPE_CAP_PRIMITIVE_RESTART_FOR_PATCHES:
>       case PIPE_CAP_TGSI_VOTE:
>       case PIPE_CAP_MAX_WINDOW_RECTANGLES:
> @@ -447,6 +446,8 @@ static int si_get_param(struct pipe_screen* pscreen, enum 
> pipe_cap param)
>       case PIPE_CAP_MULTI_DRAW_INDIRECT_PARAMS:
>               return sscreen->has_draw_indirect_multi;
>  
> +     case PIPE_CAP_CULL_DISTANCE:
> +             return 1;
>       case PIPE_CAP_MAX_SHADER_PATCH_VARYINGS:
>               return 30;

Hm, shouldn't the PIPE_CAP_CULL_DISTANCE just be moved up to the long list of
supported features with boolean caps at the beginning of si_get_param()?
(Instead of creating another "true segment"?)

Cheers,
Kai

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to