Mesa (master): microsoft/compiler: do not lower away 64-bit ffma

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Fri Jan 8 15:18:22 UTC 2021


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

Author: Erik Faye-Lund <erik.faye-lund at collabora.com>
Date:   Wed Jan  6 08:47:50 2021 +0100

microsoft/compiler: do not lower away 64-bit ffma

In DXIL, the FMA instruction only supports 64-bit operations. However,
back when we implemented support for this, there were only a single
switch for lowering all ffma instructions, so we couldn't easily use it.

But now that there's separate flags to lower ffma on 16, 32 and 64 bit,
we can lower 16 and 32 bit ffmas, and leave 64 bit ffmas alone.

Reviewed-by: Jesse Natalie <jenatali at microsoft.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8349>

---

 src/microsoft/compiler/nir_to_dxil.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/microsoft/compiler/nir_to_dxil.c b/src/microsoft/compiler/nir_to_dxil.c
index e7a53d8398b..e072d62425c 100644
--- a/src/microsoft/compiler/nir_to_dxil.c
+++ b/src/microsoft/compiler/nir_to_dxil.c
@@ -73,7 +73,6 @@ nir_options = {
    .lower_negate = true,
    .lower_ffma16 = true,
    .lower_ffma32 = true,
-   .lower_ffma64 = true,
    .lower_isign = true,
    .lower_fsign = true,
    .lower_iabs = true,



More information about the mesa-commit mailing list