[Mesa-dev] [PATCH] i965: Separate gen < 8 and gen >= 8 paths explicitly in wrap_mode()
Kenneth Graunke
kenneth at whitecape.org
Mon Jun 26 21:00:07 UTC 2017
On Monday, June 26, 2017 11:07:11 AM PDT Topi Pohjolainen wrote:
> Makes coverity happier.
>
> Fix indentation in gen >= 8 block while at it.
>
> CID: 1413020
> CC: Rafael Antognolli <rafael.antognolli at intel.com>
> Signed-off-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
> ---
> src/mesa/drivers/dri/i965/genX_state_upload.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
> index 3f8a7265db..d65b468863 100644
> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
> +++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
> @@ -4455,9 +4455,8 @@ translate_wrap_mode(struct brw_context *brw, GLenum wrap, bool using_nearest)
> *
> * Gen8+ supports this natively.
> */
> - return TCM_HALF_BORDER;
> -#endif
> -
> + return TCM_HALF_BORDER;
> +#else
> /* On Gen4-7.5, we clamp the coordinates in the fragment shader
> * and set clamp_border here, which gets the result desired.
> * We just use clamp(_to_edge) for nearest, because for nearest
> @@ -4468,6 +4467,7 @@ translate_wrap_mode(struct brw_context *brw, GLenum wrap, bool using_nearest)
> return TCM_CLAMP;
> else
> return TCM_CLAMP_BORDER;
> +#endif
> case GL_CLAMP_TO_EDGE:
> return TCM_CLAMP;
> case GL_CLAMP_TO_BORDER:
>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170626/c0d40ba1/attachment.sig>
More information about the mesa-dev
mailing list