[Mesa-dev] [PATCH] i965: Drop Gen7+ nonsense from brw_ff_gs.c.

Timothy Arceri tarceri at itsqueeze.com
Tue Aug 22 22:52:02 UTC 2017


Yeah I've noticed this before.

Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>

On 23/08/17 07:58, Kenneth Graunke wrote:
> brw_ff_gs.c is about using the geometry shader to implement things
> that the fixed function ought to do, but doesn't on old hardware.
> 
> Gen7+ does not need this.  We should drop the misleading comment
> about Gen7 not using geometry shaders.
> ---
>   src/mesa/drivers/dri/i965/brw_ff_gs.c | 7 +++----
>   1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_ff_gs.c b/src/mesa/drivers/dri/i965/brw_ff_gs.c
> index b7b4b716011..a3919524df1 100644
> --- a/src/mesa/drivers/dri/i965/brw_ff_gs.c
> +++ b/src/mesa/drivers/dri/i965/brw_ff_gs.c
> @@ -170,6 +170,8 @@ brw_ff_gs_populate_key(struct brw_context *brw,
>   
>      struct gl_context *ctx = &brw->ctx;
>   
> +   assert(brw->gen < 7);
> +
>      memset(key, 0, sizeof(*key));
>   
>      /* BRW_NEW_VS_PROG_DATA (part of VUE map) */
> @@ -187,10 +189,7 @@ brw_ff_gs_populate_key(struct brw_context *brw,
>         key->pv_first = true;
>      }
>   
> -   if (brw->gen >= 7) {
> -      /* On Gen7 and later, we don't use GS (yet). */
> -      key->need_gs_prog = false;
> -   } else if (brw->gen == 6) {
> +   if (brw->gen == 6) {
>         /* On Gen6, GS is used for transform feedback. */
>         /* BRW_NEW_TRANSFORM_FEEDBACK */
>         if (_mesa_is_xfb_active_and_unpaused(ctx)) {
> 


More information about the mesa-dev mailing list