On 11/17/2014 01:42 PM, Marek Olšák wrote:
> From: Marek Olšák <marek.ol...@amd.com>
>
> Cc: 10.4 <mesa-sta...@lists.freedesktop.org>
> ---
> src/gallium/drivers/radeonsi/si_state_draw.c | 25 +++++++++++++------------
> 1 file changed, 13 insertions(+), 12 deletions(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
> b/src/gallium/drivers/radeonsi/si_state_draw.c
> index d5b27e7..f108282 100644
> --- a/src/gallium/drivers/radeonsi/si_state_draw.c
> +++ b/src/gallium/drivers/radeonsi/si_state_draw.c
> @@ -248,20 +248,21 @@ static void si_shader_ps(struct si_shader *shader)
> for (i = 0; i < info->num_inputs; i++) {
> switch (info->input_semantic_name[i]) {
> case TGSI_SEMANTIC_POSITION:
So... now this is a switch-statement with just one case? That seems a
little weird...
> - if (info->input_interpolate_loc[i] ==
> - TGSI_INTERPOLATE_LOC_CENTROID) {
> - /* SPI_BARYC_CNTL.POS_FLOAT_LOCATION
> - * Possible vaules:
> - * 0 -> Position = pixel center (default)
> - * 1 -> Position = pixel centroid
> - * 2 -> Position = iterated sample number XXX:
> - * What does this mean?
> - */
> + /* SPI_BARYC_CNTL.POS_FLOAT_LOCATION
> + * Possible vaules:
> + * 0 -> Position = pixel center (default)
> + * 1 -> Position = pixel centroid
> + * 2 -> Position = at sample position
> + */
> + switch (info->input_interpolate_loc[i]) {
> + case TGSI_INTERPOLATE_LOC_CENTROID:
> spi_baryc_cntl |=
> S_0286E0_POS_FLOAT_LOCATION(1);
> + break;
> + case TGSI_INTERPOLATE_LOC_SAMPLE:
> + spi_baryc_cntl |=
> S_0286E0_POS_FLOAT_LOCATION(2);
> + break;
> }
> - /* Fall through */
> - case TGSI_SEMANTIC_FACE:
> - continue;
> + break;
> }
> }
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
