[Mesa-dev] [PATCH] i965/blorp: Correct type of src_format in call to intel_miptree_texture_aux_usage

Jordan Justen jordan.l.justen at intel.com
Thu Aug 10 23:34:25 UTC 2017


Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>

On 2017-08-09 15:52:30, Scott D Phillips wrote:
> intel_miptree_texture_aux_usage() takes an isl_format, but we are
> passing a mesa_format. clang warns:
> 
>  brw_blorp.c:305:52: warning: implicit conversion from enumeration
>     type 'mesa_format' to different enumeration type
>     'enum isl_format' [-Wenum-conversion]
>        intel_miptree_texture_aux_usage(brw, src_mt, src_format);
>        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~              ^~~~~~~~~~
> 
> Fixes: fc1639e46d ("i965/blorp: Use texture/render_aux_usage for blits")
> ---
>  src/mesa/drivers/dri/i965/brw_blorp.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c
> index 446f507619..d8e48064e3 100644
> --- a/src/mesa/drivers/dri/i965/brw_blorp.c
> +++ b/src/mesa/drivers/dri/i965/brw_blorp.c
> @@ -301,8 +301,9 @@ brw_blorp_blit_miptrees(struct brw_context *brw,
>        src_format = dst_format = MESA_FORMAT_R_FLOAT32;
>     }
>  
> +   enum isl_format src_isl_format = brw_isl_format_for_mesa_format(src_format);
>     enum isl_aux_usage src_aux_usage =
> -      intel_miptree_texture_aux_usage(brw, src_mt, src_format);
> +      intel_miptree_texture_aux_usage(brw, src_mt, src_isl_format);
>     /* We do format workarounds for some depth formats so we can't reliably
>      * sample with HiZ.  One of these days, we should fix that.
>      */
> -- 
> 2.13.3
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list