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

glsl: simplify loop in varying_matches::assign_locations()

Browse Source 
The use of break/continue was kind of weird/confusing.

Reviewed-by: Timothy Arceri <tarceri@itsqueeze.com>
This commit is contained in:
Brian Paul
2017-12-15 14:27:55 -07:00
parent 47b4183c92
commit cd7705de44
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/compiler/glsl/link_varyings.cpp
View File

@@ -1834,13 +1834,13 @@ varying_matches::assign_locations(struct gl_shader_program *prog,
const uint64_t slot_mask = ((1ull << slots) - 1) << (*location / 4u); const uint64_t slot_mask = ((1ull << slots) - 1) << (*location / 4u);
assert(slots > 0); assert(slots > 0);
if (reserved_slots & slot_mask) {
*location = ALIGN(*location + 1, 4); if ((reserved_slots & slot_mask) == 0) {
slot_end = *location + num_components - 1; break;
continue;
} }
break; *location = ALIGN(*location + 1, 4);
slot_end = *location + num_components - 1;
} }
if (!var->data.patch && slot_end >= MAX_VARYING * 4u) { if (!var->data.patch && slot_end >= MAX_VARYING * 4u) {