diff options
| author | Matt Turner <[email protected]> | 2017-08-25 19:22:51 -0700 |
|---|---|---|
| committer | Matt Turner <[email protected]> | 2017-08-29 15:20:57 -0700 |
| commit | dff75c7175d7a06cdeafa5c119f3749e84e2561b (patch) | |
| tree | 93ad5dbde2dd754e72307abc9055935f06a46f04 /src/intel/compiler | |
| parent | 35902f47ebdf49fa79311c009d8710ac04067527 (diff) | |
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]>
Diffstat (limited to 'src/intel/compiler')
| -rw-r--r-- | src/intel/compiler/brw_eu_validate.c | 8 |
1 files 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 |