OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

nir/gl_nir_linker: Call add_vars_with_modes once for GL_PROGRAM_INPUT

Browse Source 
Now that nir_foreach_variable_with_modes can handle multiple modes at
one time, we can simplify things a bit and only walk the list once.

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Reviewed-by: Timothy Arceri <tarceri@itsqueeze.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5966>
This commit is contained in:
Jason Ekstrand
2020-07-21 11:55:27 -05:00
committed by Marge Bot
parent 789ec95dcd
commit fc1363cc60
1 changed files with 11 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
src/compiler/glsl/gl_nir_linker.c
View File

@@ -311,12 +311,12 @@ add_shader_variable(const struct gl_context *ctx,
}
static bool
add_vars_with_mode(const struct gl_context *ctx,
struct gl_shader_program *prog, struct set *resource_set,
nir_shader *nir, nir_variable_mode mode,
unsigned stage, GLenum programInterface)
add_vars_with_modes(const struct gl_context *ctx,
struct gl_shader_program *prog, struct set *resource_set,
nir_shader *nir, nir_variable_mode modes,
unsigned stage, GLenum programInterface)
{
nir_foreach_variable_with_modes(var, nir, mode) {
nir_foreach_variable_with_modes(var, nir, modes) {
if (var->data.how_declared == nir_var_hidden)
continue;
@@ -405,18 +405,14 @@ add_interface_variables(const struct gl_context *ctx,
switch (programInterface) {
case GL_PROGRAM_INPUT: {
bool result = add_vars_with_mode(ctx, prog, resource_set,
nir, nir_var_shader_in,
stage, programInterface);
result &= add_vars_with_mode(ctx, prog, resource_set,
nir, nir_var_system_value,
stage, programInterface);
return result;
return add_vars_with_modes(ctx, prog, resource_set,
nir, nir_var_shader_in | nir_var_system_value,
stage, programInterface);
}
case GL_PROGRAM_OUTPUT:
return add_vars_with_mode(ctx, prog, resource_set,
nir, nir_var_shader_out,
stage, programInterface);
return add_vars_with_modes(ctx, prog, resource_set,
nir, nir_var_shader_out,
stage, programInterface);
default:
assert("!Should not get here");
break;