[Mesa-dev] i965: Remove duplicate checks

Juha-Pekka Heikkila juhapekka.heikkila at gmail.com
Fri Feb 24 13:10:20 UTC 2017


On both sides of the comment on what's being checked are exactly
same checks in different order. I took out ones before the comment
so comment can stay in place.

Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
---
 src/mesa/drivers/dri/i965/brw_eu_validate.c | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_eu_validate.c b/src/mesa/drivers/dri/i965/brw_eu_validate.c
index 64615af..3655dde 100644
--- a/src/mesa/drivers/dri/i965/brw_eu_validate.c
+++ b/src/mesa/drivers/dri/i965/brw_eu_validate.c
@@ -407,18 +407,6 @@ general_restrictions_based_on_operand_types(const struct gen_device_info *devinf
    unsigned exec_size = 1 << brw_inst_exec_size(devinfo, inst);
    struct string error_msg = { .str = NULL, .len = 0 };
 
-   if (num_sources == 3)
-      return (struct string){};
-
-   if (inst_is_send(devinfo, inst))
-      return (struct string){};
-
-   if (exec_size == 1)
-      return (struct string){};
-
-   if (desc->ndst == 0)
-      return (struct string){};
-
    /* The PRMs say:
     *
     *    Where n is the largest element size in bytes for any source or
-- 
2.7.4



More information about the mesa-dev mailing list