From: Samuel Pitoiset Date: Wed, 14 Feb 2018 11:03:55 +0000 (+0100) Subject: ac/shader: be a little smarter when scanning vertex buffers X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=61a4fc3ecc3d7740d2b9122c502144e69fe3060c;p=mesa.git ac/shader: be a little smarter when scanning vertex buffers Although meta shaders don't use any vertex buffers, there is no behaviour change but I think it's better to do this. Though, this saves two user SGPRs for push constants inlining or something else. Signed-off-by: Samuel Pitoiset Reviewed-by: Bas Nieuwenhuizen --- diff --git a/src/amd/common/ac_shader_info.c b/src/amd/common/ac_shader_info.c index 3b0887995d3..f6cdd347035 100644 --- a/src/amd/common/ac_shader_info.c +++ b/src/amd/common/ac_shader_info.c @@ -179,6 +179,16 @@ gather_info_block(const nir_shader *nir, const nir_block *block, } } +static void +gather_info_input_decl_vs(const nir_shader *nir, const nir_variable *var, + struct ac_shader_info *info) +{ + int idx = var->data.location; + + if (idx >= VERT_ATTRIB_GENERIC0 && idx <= VERT_ATTRIB_GENERIC15) + info->vs.has_vertex_buffers = true; +} + static void gather_info_input_decl_ps(const nir_shader *nir, const nir_variable *var, struct ac_shader_info *info) @@ -197,7 +207,7 @@ gather_info_input_decl(const nir_shader *nir, const nir_variable *var, { switch (nir->info.stage) { case MESA_SHADER_VERTEX: - info->vs.has_vertex_buffers = true; + gather_info_input_decl_vs(nir, var, info); break; case MESA_SHADER_FRAGMENT: gather_info_input_decl_ps(nir, var, info);