glsl: drop the dce of global vars from GLSL IR linker
authorTimothy Arceri <tarceri@itsqueeze.com>
Wed, 5 Apr 2023 02:30:24 +0000 (12:30 +1000)
committerMarge Bot <emma+marge@anholt.net>
Thu, 6 Jul 2023 01:21:06 +0000 (01:21 +0000)
commita8534c7b4cfc0f8450da2c03d8ffb74cdafee4c6
tree67537c35f4a181a2f69eedba7c195962c3ccd37b
parenta8c485efee4f7fb50faff0c56b59dc3c9d66dd41
glsl: drop the dce of global vars from GLSL IR linker

All this does is compilcate things such as forcing us to set
var->data.always_active_io in the glsl linker. Just let NIR clean
these up for us instead.

A Zink test hits a new assert but this is not a regression it just
uncovers an existing mesa bug.

Reviewed-by: Emma Anholt <emma@anholt.net>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22846>
src/compiler/glsl/glsl_parser_extras.cpp
src/gallium/drivers/zink/ci/zink-radv-vangogh-fails.txt