diff options
| author | Juan A. Suarez Romero <jasuarez@igalia.com> | 2021-03-30 11:00:01 +0200 |
|---|---|---|
| committer | Marge Bot <eric+marge@anholt.net> | 2021-03-30 14:15:43 +0000 |
| commit | 1f90d517494021970576d826df64e5b6625361b8 (patch) | |
| tree | eea779a3977ace5a938bad5a1fe27493fb693220 | |
| parent | cc1f070a27d78d9735f471714607a16e40f78cff (diff) | |
v3dv: fix unused value
Do not assign to a variable that won't be used.
Fixes CID#1468098 "Unused value (UNUSED_VALUE)".
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/9910>
| -rw-r--r-- | src/broadcom/vulkan/v3dv_cmd_buffer.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/broadcom/vulkan/v3dv_cmd_buffer.c b/src/broadcom/vulkan/v3dv_cmd_buffer.c index 1b5c93927dd..4b5b7b552f9 100644 --- a/src/broadcom/vulkan/v3dv_cmd_buffer.c +++ b/src/broadcom/vulkan/v3dv_cmd_buffer.c @@ -4314,10 +4314,9 @@ cmd_buffer_emit_pre_draw(struct v3dv_cmd_buffer *cmd_buffer) * an active job. In that case, create a new job continuing the current * subpass. */ - struct v3dv_job *job = cmd_buffer->state.job; - if (!job) { - job = v3dv_cmd_buffer_subpass_resume(cmd_buffer, - cmd_buffer->state.subpass_idx); + if (!cmd_buffer->state.job) { + v3dv_cmd_buffer_subpass_resume(cmd_buffer, + cmd_buffer->state.subpass_idx); } /* Restart single sample job for MSAA pipeline if needed */ @@ -4326,7 +4325,7 @@ cmd_buffer_emit_pre_draw(struct v3dv_cmd_buffer *cmd_buffer) /* If the job is configured to flush on every draw call we need to create * a new job now. */ - job = cmd_buffer_pre_draw_split_job(cmd_buffer); + struct v3dv_job *job = cmd_buffer_pre_draw_split_job(cmd_buffer); job->draw_count++; /* GL shader state binds shaders, uniform and vertex attribute state. The |