v3dv: fix assertion

Ensure subpass_idx has a valid value; we use "-1" as invalid one.

Fixes CID#1468096 "Macro compares unsigned to 0 (NO_EFFECT)"

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10203>
This commit is contained in:
Juan A. Suarez Romero 2021-04-13 10:00:14 +02:00 committed by Marge Bot
parent d69d8817c9
commit cbb1e2dcac

View file

@ -2680,7 +2680,7 @@ static struct v3dv_job *
cmd_buffer_subpass_split_for_barrier(struct v3dv_cmd_buffer *cmd_buffer,
bool is_bcl_barrier)
{
assert(cmd_buffer->state.subpass_idx >= 0);
assert(cmd_buffer->state.subpass_idx != -1);
v3dv_cmd_buffer_finish_job(cmd_buffer);
struct v3dv_job *job =
v3dv_cmd_buffer_subpass_resume(cmd_buffer,