Mesa (master): radv: Call nir_lower_constant_initializers.
Bas Nieuwenhuizen
bnieuwenhuizen at kemper.freedesktop.org
Thu Jan 12 20:49:33 UTC 2017
Module: Mesa
Branch: master
Commit: 65cbb993d33976d9ee24eff01ade8ed9013617ca
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=65cbb993d33976d9ee24eff01ade8ed9013617ca
Author: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Date: Sun Jan 8 23:17:38 2017 +0100
radv: Call nir_lower_constant_initializers.
Port of c5d664f9dc2d281c74844cef36ecb9f5862a8f6a
"anv/pipeline: Call nir_lower_constant_initializers"
Signed-off-by: Bas Nieuwenhuizen <basni at google.com>
Cc: <mesa-stable at lists.freedesktop.org>
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>
---
src/amd/vulkan/radv_pipeline.c | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 03603f5..d818938 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b/src/amd/vulkan/radv_pipeline.c
@@ -205,6 +205,13 @@ radv_shader_compile_to_nir(struct radv_device *device,
free(spec_entries);
+ /* We have to lower away local constant initializers right before we
+ * inline functions. That way they get properly initialized at the top
+ * of the function and not at the top of its caller.
+ */
+ nir_lower_constant_initializers(nir, nir_var_local);
+ nir_validate_shader(nir);
+
nir_lower_returns(nir);
nir_validate_shader(nir);
@@ -224,6 +231,12 @@ radv_shader_compile_to_nir(struct radv_device *device,
nir_var_system_value);
nir_validate_shader(nir);
+ /* Now that we've deleted all but the main function, we can go ahead and
+ * lower the rest of the constant initializers.
+ */
+ nir_lower_constant_initializers(nir, ~0);
+ nir_validate_shader(nir);
+
nir_lower_system_values(nir);
nir_validate_shader(nir);
}
More information about the mesa-commit
mailing list