[Mesa-dev] [PATCH] anv: Fix a harmless overflow warning

Anuj Phogat anuj.phogat at gmail.com
Wed Jun 15 00:09:33 UTC 2016


On Tue, Jun 14, 2016 at 4:22 PM, Chad Versace <chad.versace at intel.com> wrote:
> anv_pipeline_binding::index is a uint8_t, but some code assigned to it
> UINT16_MAX.
> ---
>  src/intel/vulkan/anv_pipeline.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c
> index 60b7c6b..b41e11e 100644
> --- a/src/intel/vulkan/anv_pipeline.c
> +++ b/src/intel/vulkan/anv_pipeline.c
> @@ -664,7 +664,7 @@ anv_pipeline_compile_fs(struct anv_pipeline *pipeline,
>           rt_bindings[0] = (struct anv_pipeline_binding) {
>              .set = ANV_DESCRIPTOR_SET_COLOR_ATTACHMENTS,
>              .binding = 0,
> -            .index = UINT16_MAX,
> +            .index = UINT8_MAX,
>           };
>           num_rts = 1;
>        }
> --
> 2.9.0.rc2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev

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


More information about the mesa-dev mailing list