Mesa (master): radeonsi: Enable NIR's lower_fmod option.

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Jun 5 23:56:31 UTC 2019


Module: Mesa
Branch: master
Commit: f4d4c426084127d88aba11f22b4f33ff93700963
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f4d4c426084127d88aba11f22b4f33ff93700963

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Mon Jun  3 13:40:05 2019 -0700

radeonsi: Enable NIR's lower_fmod option.

Currently, st/mesa is always calling the GLSL IR lower_instructions()
pass with MOD_TO_FLOOR set, so mod operations will be lowered before
ever reaching NIR.  This enables the same lowering at the NIR level,
which will let me shut off the GLSL IR path for NIR-based drivers.

The AMD NIR backend also has code to handle fmod, so we could
potentially skip this and still be fine.  I don't have an opinion
on that.

Reviewed-by: Marek Olšák <marek.olsak at amd.com>

---

 src/gallium/drivers/radeonsi/si_get.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/gallium/drivers/radeonsi/si_get.c b/src/gallium/drivers/radeonsi/si_get.c
index 8217c10d9c9..40a54b34c4f 100644
--- a/src/gallium/drivers/radeonsi/si_get.c
+++ b/src/gallium/drivers/radeonsi/si_get.c
@@ -486,6 +486,7 @@ static const struct nir_shader_compiler_options nir_options = {
 	.lower_fdiv = true,
 	.lower_sub = true,
 	.lower_ffma = true,
+	.lower_fmod = true,
 	.lower_pack_snorm_2x16 = true,
 	.lower_pack_snorm_4x8 = true,
 	.lower_pack_unorm_2x16 = true,




More information about the mesa-commit mailing list