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