[Mesa-dev] [PATCH] draw/gs: fix point size outputs from geometry shader.

Brian Paul brianp at vmware.com
Mon Mar 25 14:18:47 UTC 2019


On 03/24/2019 11:14 PM, Dave Airlie wrote:
> From: Dave Airlie <airlied at redhat.com>
> 
> If the geom shader emits a point size we failed to find it here,
> use the correct API to look it up.
> 
> Fixes:
> tests/spec/glsl-1.50/execution/geometry/point-size-out.shader_test
> ---
>   src/gallium/auxiliary/draw/draw_pipe_wide_point.c | 9 +--------
>   1 file changed, 1 insertion(+), 8 deletions(-)
> 
> diff --git a/src/gallium/auxiliary/draw/draw_pipe_wide_point.c b/src/gallium/auxiliary/draw/draw_pipe_wide_point.c
> index 1329ab4e7c0..e9bbb67a958 100644
> --- a/src/gallium/auxiliary/draw/draw_pipe_wide_point.c
> +++ b/src/gallium/auxiliary/draw/draw_pipe_wide_point.c
> @@ -266,14 +266,7 @@ widepoint_first_point(struct draw_stage *stage,
>      wide->psize_slot = -1;
>      if (rast->point_size_per_vertex) {
>         /* find PSIZ vertex output */
> -      const struct draw_vertex_shader *vs = draw->vs.vertex_shader;
> -      uint i;
> -      for (i = 0; i < vs->info.num_outputs; i++) {
> -         if (vs->info.output_semantic_name[i] == TGSI_SEMANTIC_PSIZE) {
> -            wide->psize_slot = i;
> -            break;
> -         }
> -      }
> +      wide->psize_slot = draw_find_shader_output(draw, TGSI_SEMANTIC_PSIZE, 0);
>      }
>      
>      stage->point( stage, header );
> 

Reviewed-by: Brian Paul <brianp at vmware.com>


More information about the mesa-dev mailing list