[Mesa-dev] [PATCH 2/2] i965: Start using HIZ for Z16 textures.

[Kenneth Graunke]

On 12/23/2012 12:16 PM, Eric Anholt wrote:
> I had left this out for a long time because it regressed some
> depthstencil-render-miplevels cases when it was enabled.  Now that the
> bugs causing those are fixed, there's nothing stopping us.
>
> Improves glbenchmark 2.1 offscreen performance by 7.3% +/- 2.8% (n=10).
> ---
>   src/mesa/drivers/dri/i965/brw_vtbl.c |    1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vtbl.c b/src/mesa/drivers/dri/i965/brw_vtbl.c
> index 237ec1e..cb25be9 100644
> --- a/src/mesa/drivers/dri/i965/brw_vtbl.c
> +++ b/src/mesa/drivers/dri/i965/brw_vtbl.c
> @@ -239,6 +239,7 @@ static bool brw_is_hiz_depth_format(struct intel_context *intel,
>      case MESA_FORMAT_Z32_FLOAT_X24S8:
>      case MESA_FORMAT_X8_Z24:
>      case MESA_FORMAT_S8_Z24:
> +   case MESA_FORMAT_Z16:
>         return true;
>      default:
>         return false;

LGTM.

Both are:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>