Mesa (main): intel/compiler: remove redundant code from fs_visitor::run_*

[GitLab Mirror]

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

Author: Marcin Ślusarz <marcin.slusarz at intel.com>
Date:   Fri Feb 18 13:16:07 2022 +0100

intel/compiler: remove redundant code from fs_visitor::run_*

Reviewed-by: Caio Oliveira <caio.oliveira at intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15079>

---

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

diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp
index 240a53b9fee..359262ca489 100644
--- a/src/intel/compiler/brw_fs.cpp
+++ b/src/intel/compiler/brw_fs.cpp
@@ -9217,9 +9217,6 @@ fs_visitor::run_fs(bool allow_spilling, bool do_rep_send)
       fixup_3src_null_dest();
 
       allocate_registers(allow_spilling);
-
-      if (failed)
-         return false;
    }
 
    return !failed;
@@ -9255,9 +9252,6 @@ fs_visitor::run_cs(bool allow_spilling)
    fixup_3src_null_dest();
    allocate_registers(allow_spilling);
 
-   if (failed)
-      return false;
-
    return !failed;
 }
 
@@ -9286,9 +9280,6 @@ fs_visitor::run_bs(bool allow_spilling)
    fixup_3src_null_dest();
    allocate_registers(allow_spilling);
 
-   if (failed)
-      return false;
-
    return !failed;
 }
 
@@ -9333,9 +9324,6 @@ fs_visitor::run_task(bool allow_spilling)
    fixup_3src_null_dest();
    allocate_registers(allow_spilling);
 
-   if (failed)
-      return false;
-
    return !failed;
 }
 
@@ -9380,9 +9368,6 @@ fs_visitor::run_mesh(bool allow_spilling)
    fixup_3src_null_dest();
    allocate_registers(allow_spilling);
 
-   if (failed)
-      return false;
-
    return !failed;
 }