[Mesa-dev] [PATCH] i965/gen8: Remove gen<8 checks in gen8 code

[Anuj Phogat]

On Thu, Oct 8, 2015 at 12:21 PM, Chad Versace <chad.versace at intel.com> wrote:
> Some assertions in gen8_surface_state.c checked for gen < 8.
> ---
>  src/mesa/drivers/dri/i965/gen8_surface_state.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/gen8_surface_state.c b/src/mesa/drivers/dri/i965/gen8_surface_state.c
> index e1e7704..18b8665 100644
> --- a/src/mesa/drivers/dri/i965/gen8_surface_state.c
> +++ b/src/mesa/drivers/dri/i965/gen8_surface_state.c
> @@ -221,8 +221,8 @@ gen8_emit_texture_surface_state(struct brw_context *brw,
>         * "When Auxiliary Surface Mode is set to AUX_CCS_D or AUX_CCS_E, HALIGN
>         *  16 must be used."
>         */
> -      assert(brw->gen < 9 || mt->halign == 16);
> -      assert(brw->gen < 8 || mt->num_samples > 1 || mt->halign == 16);
> +      if (brw->gen >= 9 || mt->num_samples == 1)
> +         assert(mt->halign == 16);
>     }
>
>     const uint32_t surf_type = translate_tex_target(target);
> @@ -470,8 +470,8 @@ gen8_update_renderbuffer_surface(struct brw_context *brw,
>         * "When Auxiliary Surface Mode is set to AUX_CCS_D or AUX_CCS_E, HALIGN
>         *  16 must be used."
>         */
> -      assert(brw->gen < 9 || mt->halign == 16);
> -      assert(brw->gen < 8 || mt->num_samples > 1 || mt->halign == 16);
> +      if (brw->gen >= 9 || mt->num_samples == 1)
> +         assert(mt->halign == 16);
>     }
>
>     uint32_t *surf = allocate_surface_state(brw, &offset, surf_index);
> --
> 2.5.0.342.g44e0223
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>