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-Tag: upstream/18.1.0~1682 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=61a4fc3ecc3d7740d2b9122c502144e69fe3060c;p=platform%2Fupstream%2Fmesa.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 3b08879..f6cdd34 100644 --- a/src/amd/common/ac_shader_info.c +++ b/src/amd/common/ac_shader_info.c @@ -180,6 +180,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);