<p dir="ltr"><br>
On Jun 14, 2016 4:23 PM, "Chad Versace" <<a href="mailto:chad.versace@intel.com">chad.versace@intel.com</a>> wrote:<br>
><br>
> anv_pipeline_binding::index is a uint8_t, but some code assigned to it<br>
> UINT16_MAX.<br>
> ---<br>
>  src/intel/vulkan/anv_pipeline.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c<br>
> index 60b7c6b..b41e11e 100644<br>
> --- a/src/intel/vulkan/anv_pipeline.c<br>
> +++ b/src/intel/vulkan/anv_pipeline.c<br>
> @@ -664,7 +664,7 @@ anv_pipeline_compile_fs(struct anv_pipeline *pipeline,<br>
>           rt_bindings[0] = (struct anv_pipeline_binding) {<br>
>              .set = ANV_DESCRIPTOR_SET_COLOR_ATTACHMENTS,<br>
>              .binding = 0,<br>
> -            .index = UINT16_MAX,<br>
> +            .index = UINT8_MAX,</p>
<p dir="ltr">I believe we have a descriptive #define specifically for render targets. Probably better to use that.</p>
<p dir="ltr">>           };<br>
>           num_rts = 1;<br>
>        }<br>
> --<br>
> 2.9.0.rc2<br>
><br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>