summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
4f874a8)
The "Thread *th" variable is used only on a few control paths, and it is never
reused. Might as well just calculate it on the fly. Also, it's recalculated and
masked by another local variable at a deeper level of nesting.
- Thread *th = get_thread(curr);
-
bool updated = false;
if (curr->is_read()) {
bool updated = false;
if (curr->is_read()) {
- updated = process_read(curr, th, second_part_of_rmw);
+ updated = process_read(curr, get_thread(curr), second_part_of_rmw);
}
if (curr->is_write()) {
}
if (curr->is_write()) {
}
mo_graph->commitChanges();
}
mo_graph->commitChanges();
- th->set_return_value(VALUE_NONE);
+ get_thread(curr)->set_return_value(VALUE_NONE);