From dff75c7175d7a06cdeafa5c119f3749e84e2561b Mon Sep 17 00:00:00 2001 From: Matt Turner Date: Fri, 25 Aug 2017 19:22:51 -0700 Subject: [PATCH] i965: Drop unnecessary conditional Clang doesn't realize that 0 and 1 are the only possibilities, a thinks lots of variables might be uninitialized. Reviewed-by: Emil Velikov Reviewed-by: Eric Engestrom --- src/intel/compiler/brw_eu_validate.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/intel/compiler/brw_eu_validate.c b/src/intel/compiler/brw_eu_validate.c index 249342fb9dc..b24b38170e0 100644 --- a/src/intel/compiler/brw_eu_validate.c +++ b/src/intel/compiler/brw_eu_validate.c @@ -555,7 +555,7 @@ general_restrictions_on_region_parameters(const struct gen_device_info *devinfo, if (i == 0) { DO_SRC(0); - } else if (i == 1) { + } else { DO_SRC(1); } #undef DO_SRC @@ -758,7 +758,7 @@ region_alignment_rules(const struct gen_device_info *devinfo, if (i == 0) { DO_SRC(0); - } else if (i == 1) { + } else { DO_SRC(1); } #undef DO_SRC @@ -950,7 +950,7 @@ region_alignment_rules(const struct gen_device_info *devinfo, if (i == 0) { DO_SRC(0); - } else if (i == 1) { + } else { DO_SRC(1); } #undef DO_SRC @@ -1011,7 +1011,7 @@ region_alignment_rules(const struct gen_device_info *devinfo, if (i == 0) { DO_SRC(0); - } else if (i == 1) { + } else { DO_SRC(1); } #undef DO_SRC -- 2.30.2