Mesa (master): intel: Set int64_options to ~0 when lowering 64b ops

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Thu Jul 30 17:09:35 UTC 2020


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

Author: Boris Brezillon <boris.brezillon at collabora.com>
Date:   Mon Jul 13 20:57:04 2020 +0200

intel: Set int64_options to ~0 when lowering 64b ops

That's more future proof than setting each bit manually. Looks like we
already miss nir_lower_ufind_msb64 because of that.

Signed-off-by: Boris Brezillon <boris.brezillon at collabora.com>
Reviewed-by: Matt Turner <mattst88 at gmail.com>
Acked-by: Jason Ekstrand <jason at jlekstrand.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5588>

---

 src/intel/compiler/brw_compiler.c | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/src/intel/compiler/brw_compiler.c b/src/intel/compiler/brw_compiler.c
index c26b6eb3bbc..a75e86346bf 100644
--- a/src/intel/compiler/brw_compiler.c
+++ b/src/intel/compiler/brw_compiler.c
@@ -146,15 +146,7 @@ brw_compiler_create(void *mem_ctx, const struct gen_device_info *devinfo)
       nir_lower_ddiv;
 
    if (!devinfo->has_64bit_float || (INTEL_DEBUG & DEBUG_SOFT64)) {
-      int64_options |= nir_lower_mov64 |
-                       nir_lower_icmp64 |
-                       nir_lower_iadd64 |
-                       nir_lower_iabs64 |
-                       nir_lower_ineg64 |
-                       nir_lower_logic64 |
-                       nir_lower_minmax64 |
-                       nir_lower_shift64 |
-                       nir_lower_extract64;
+      int64_options |= (nir_lower_int64_options)~0;
       fp64_options |= nir_lower_fp64_full_software;
    }
 



More information about the mesa-commit mailing list