diff options
author | Kenneth Graunke <[email protected]> | 2019-06-03 13:18:55 -0700 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2019-06-05 16:45:12 -0700 |
commit | c7d1b52a2c2bf8dc8066db8a7adb02241ccf9e2d (patch) | |
tree | b61f57c9f9511119df845cac03d478134681eb5b /src/gallium | |
parent | edd45af9ba1952e2656af5772c5c5704b1884eba (diff) |
nir: Combine lower_fmod16/32 back into a single lower_fmod.
We originally had a single lower_fmod option. In commit 2ab2d2e5, Sam
split 32 and 64-bit lowering into separate flags, with the rationale
that some drivers might want different options there. This left 16-bit
unhandled, so Iago added a lower_fmod16 option in commit ca31df6f.
Now that lower_fmod64 is gone (in favor of nir_lower_doubles and
nir_lower_dmod), we re-combine lower_fmod16 and lower_fmod32 into a
single lower_fmod flag again. I'm not aware of any hardware which
need lowering for one bitsize and not the other.
Reviewed-by: Marek Olšák <[email protected]>
Diffstat (limited to 'src/gallium')
-rw-r--r-- | src/gallium/drivers/freedreno/a2xx/ir2_nir.c | 2 | ||||
-rw-r--r-- | src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 | ||||
-rw-r--r-- | src/gallium/drivers/panfrost/midgard/midgard_compile.h | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/src/gallium/drivers/freedreno/a2xx/ir2_nir.c b/src/gallium/drivers/freedreno/a2xx/ir2_nir.c index 5cfb831b6e2..36f3a679ff6 100644 --- a/src/gallium/drivers/freedreno/a2xx/ir2_nir.c +++ b/src/gallium/drivers/freedreno/a2xx/ir2_nir.c @@ -32,7 +32,7 @@ static const nir_shader_compiler_options options = { .lower_fpow = true, .lower_flrp32 = true, - .lower_fmod32 = true, + .lower_fmod = true, .lower_fdiv = true, .lower_fceil = true, .fuse_ffma = true, diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c index 09f8c4135e5..527d5a8108a 100644 --- a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c @@ -905,7 +905,7 @@ static const nir_shader_compiler_options nir_options = { .lower_fpow = false, .lower_fsat = false, .lower_fsqrt = false, // TODO: only before gm200 - .lower_fmod32 = true, + .lower_fmod = true, .lower_bitfield_extract = false, .lower_bitfield_extract_to_shifts = false, .lower_bitfield_insert = false, diff --git a/src/gallium/drivers/panfrost/midgard/midgard_compile.h b/src/gallium/drivers/panfrost/midgard/midgard_compile.h index ba9e74adef3..2ee0dfbb1ae 100644 --- a/src/gallium/drivers/panfrost/midgard/midgard_compile.h +++ b/src/gallium/drivers/panfrost/midgard/midgard_compile.h @@ -91,7 +91,7 @@ static const nir_shader_compiler_options midgard_nir_options = { .lower_flrp32 = true, .lower_flrp64 = true, .lower_ffract = true, - .lower_fmod32 = true, + .lower_fmod = true, .lower_fdiv = true, .lower_idiv = true, .lower_isign = true, |