OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

i965: Drop unnecessary conditional

Browse Source 
Clang doesn't realize that 0 and 1 are the only possibilities, a thinks
lots of variables might be uninitialized.

Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
This commit is contained in:
Matt Turner
2017-08-25 19:22:51 -07:00
parent 35902f47eb
commit dff75c7175
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/intel/compiler/brw_eu_validate.c
View File

@@ -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