diff options
author | Caio Marcelo de Oliveira Filho <[email protected]> | 2019-08-21 11:08:48 -0700 |
---|---|---|
committer | Caio Marcelo de Oliveira Filho <[email protected]> | 2019-09-10 14:36:46 -0700 |
commit | 664e4a610dc8c0f2adc50de645a07cf4e2b622fd (patch) | |
tree | 298d803526806d1f71e5109e11e443e8a2774e98 /src/mesa/drivers/dri | |
parent | eea3aa25aa20599578a20fb3256fdb90882abb2b (diff) |
glsl/nir: Fill in the Parameters in NIR linker
The parameter lists were not being created nor filled since i965
doesn't use them. In Gallium they are used for uniform handling, so
add a way to fill them.
The gl_uniform_storage struct got two new fields that let us go
- from a Parameter to the matching UniformStorage and,
- from the variable to the *first* UniformStorage
without relying on names -- since they are optional for ARB_gl_spirv.
Later patches will make use of them.
v2: Do not fill parameters for i965. (Timothy)
Use uint32_t for the new attributes. (Marek)
v3: Serialize the new fields. (Timothy)
Reviewed-by: Timothy Arceri <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri')
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_link.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_link.cpp b/src/mesa/drivers/dri/i965/brw_link.cpp index e4e0b5fe330..c726551513a 100644 --- a/src/mesa/drivers/dri/i965/brw_link.cpp +++ b/src/mesa/drivers/dri/i965/brw_link.cpp @@ -273,7 +273,7 @@ brw_link_shader(struct gl_context *ctx, struct gl_shader_program *shProg) return GL_FALSE; } - if (!gl_nir_link_uniforms(ctx, shProg)) + if (!gl_nir_link_uniforms(ctx, shProg, /* fill_parameters */ false)) return GL_FALSE; gl_nir_link_assign_atomic_counter_resources(ctx, shProg); |