fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2025-12-24 13:10:10 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

glsl: remove unrequired do_lower_jumps() call

Browse source 
We were using this to remove unreachable instructions following
jumps. The previous patch allowed glsl to nir to handle these
instructions so this call is no longer needed.

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27288>
This commit is contained in:
Timothy Arceri 2024-01-26 15:37:46 +11:00 • committed by Marge Bot
parent 1391bc3721
commit 182bff5c05
1 changed files with 0 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/compiler/glsl/linker.cpp
View file

@ -2820,23 +2820,6 @@ link_shaders(struct gl_context *ctx, struct gl_shader_program *prog)
prog->_LinkedShaders[MESA_SHADER_FRAGMENT]))
goto done;
for (unsigned i = 0; i < MESA_SHADER_STAGES; i++) {
if (prog->_LinkedShaders[i] == NULL)
continue;
struct gl_linked_shader *shader = prog->_LinkedShaders[i];
exec_list *ir = shader->ir;
const struct gl_shader_compiler_options *gl_options =
&consts->ShaderCompilerOptions[i];
/* NIR cannot handle instructions after a break so we use the GLSL IR do
* lower jumps pass to clean those up for now.
*/
do_lower_jumps(ir, true, true, gl_options->EmitNoMainReturn,
gl_options->EmitNoCont);
}
done:
for (unsigned i = 0; i < MESA_SHADER_STAGES; i++) {
free(shader_list[i]);