v3d: assert if v3d_compile returns NULL
authorAlejandro Piñeiro <apinheiro@igalia.com>
Wed, 24 May 2023 08:43:55 +0000 (10:43 +0200)
committerMarge Bot <emma+marge@anholt.net>
Wed, 24 May 2023 14:19:12 +0000 (14:19 +0000)
Ideally we would like to trigger a compilation error like we do on
v3dv (VK_ERROR_UNKNOWN). But with v3d we can't really do that, as this
could happen on a draw call. Let's at least assert so debug builds
stops at this point.

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23203>

src/gallium/drivers/v3d/v3d_program.c

index 97535a1..32f0a7e 100644 (file)
@@ -407,6 +407,14 @@ v3d_get_compiled_shader(struct v3d_context *v3d,
                                         v3d_shader_debug_output,
                                         v3d,
                                         program_id, variant_id, &shader_size);
+
+                /* qpu_insts being NULL can happen if the register allocation
+                 * failed. At this point we can't really trigger an OpenGL API
+                 * error, as the final compilation could happen on the draw
+                 * call. So let's at least assert, so debug builds finish at
+                 * this point.
+                 */
+                assert(qpu_insts);
                 ralloc_steal(shader, shader->prog_data.base);
 
                 if (shader_size) {