On Jun 14, 2016 4:23 PM, "Chad Versace" <chad.vers...@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,
I believe we have a descriptive #define specifically for render targets.
Probably better to use that.
> };
> num_rts = 1;
> }
> --
> 2.9.0.rc2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
