Looks good.

Should PIPE_TEXTURE_RECT be allowed too?

Jose

----- Original Message -----
> From: Roland Scheidegger <[email protected]>
> 
> This optimized filter (when using repeat wrap modes,
> linear min/mag/mip filters, pot textures) only applies to 2d textures,
> but nothing prevented it from being used for other textures (likely
> leading to very bogus sample results).
> 
> Note: This is a candidate for the 9.0 branch.
> ---
>  src/gallium/drivers/softpipe/sp_tex_sample.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/gallium/drivers/softpipe/sp_tex_sample.c
> b/src/gallium/drivers/softpipe/sp_tex_sample.c
> index 60c1f92..bbc011e 100644
> --- a/src/gallium/drivers/softpipe/sp_tex_sample.c
> +++ b/src/gallium/drivers/softpipe/sp_tex_sample.c
> @@ -2976,6 +2976,7 @@ sp_create_sampler_variant( const struct
> pipe_sampler_state *sampler,
>  
>     case PIPE_TEX_MIPFILTER_LINEAR:
>        if (key.bits.is_pot &&
> +          key.bits.target == PIPE_TEXTURE_2D &&
>            sampler->min_img_filter == sampler->mag_img_filter &&
>            sampler->normalized_coords &&
>            sampler->wrap_s == PIPE_TEX_WRAP_REPEAT &&
> --
> 1.7.9.5
> 
_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to