Mesa (master): intel/fs: Remove redundant assert()

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Tue May 26 20:45:57 UTC 2020


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

Author: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
Date:   Tue May 19 14:30:01 2020 -0700

intel/fs: Remove redundant assert()

This is covered by the two previous similar asserts.  Each time `v` is
assigned this is asserted.

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5213>

---

 src/intel/compiler/brw_fs.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp
index e810f1fe769..0ff32148e0c 100644
--- a/src/intel/compiler/brw_fs.cpp
+++ b/src/intel/compiler/brw_fs.cpp
@@ -9127,9 +9127,6 @@ brw_compile_cs(const struct brw_compiler *compiler, void *log_data,
       }
    }
 
-   /* We should always be able to do SIMD32 for compute shaders */
-   assert(!v16 || v16->max_dispatch_width >= 32);
-
    if (!fail_msg && (min_dispatch_width > 16 || (INTEL_DEBUG & DEBUG_DO32)) &&
        max_dispatch_width >= 32) {
       /* Try a SIMD32 compile */



More information about the mesa-commit mailing list