Module: Mesa Branch: master Commit: dff75c7175d7a06cdeafa5c119f3749e84e2561b URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=dff75c7175d7a06cdeafa5c119f3749e84e2561b
Author: Matt Turner <[email protected]> Date: Fri Aug 25 19:22:51 2017 -0700 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 <[email protected]> Reviewed-by: Eric Engestrom <[email protected]> --- 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 249342fb9d..b24b38170e 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 _______________________________________________ mesa-commit mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-commit
