Mesa (master): intel: compiler: remove duplicated code

Lionel Landwerlin llandwerlin at kemper.freedesktop.org
Tue May 9 15:59:18 UTC 2017


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

Author: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Date:   Mon May  8 18:50:53 2017 +0100

intel: compiler: remove duplicated code

CID: 1399470: (Control flow issues)

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Reviewed-by: Matt Turner <mattst88 at gmail.com>

---

 src/intel/compiler/brw_eu_validate.c | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/src/intel/compiler/brw_eu_validate.c b/src/intel/compiler/brw_eu_validate.c
index f231ea038b..63b798c771 100644
--- a/src/intel/compiler/brw_eu_validate.c
+++ b/src/intel/compiler/brw_eu_validate.c
@@ -434,18 +434,6 @@ general_restrictions_based_on_operand_types(const struct gen_device_info *devinf
     * In fact, checking it would weaken testing of the other rules.
     */
 
-   if (num_sources == 3)
-      return (struct string){};
-
-   if (exec_size == 1)
-      return (struct string){};
-
-   if (inst_is_send(devinfo, inst))
-      return (struct string){};
-
-   if (desc->ndst == 0)
-      return (struct string){};
-
    unsigned dst_stride = 1 << (brw_inst_dst_hstride(devinfo, inst) - 1);
    bool dst_type_is_byte =
       brw_inst_dst_reg_type(devinfo, inst) == BRW_HW_REG_NON_IMM_TYPE_B ||




More information about the mesa-commit mailing list