Page MenuHomePhabricator

tsan: removing redundant loop in ThreadClock::release()
ClosedPublic

Authored by dfava on Feb 19 2020, 6:13 AM.

Details

Reviewers
dvyukov
Summary

The removed loop clears reused for entries at the tail of a SyncClock. The loop is redundant since those entries were already cleared by the immediately preceding loop, which iterates over all entries in the SyncClock (including the tail entries).

Diff Detail

Event Timeline

dfava created this revision.Feb 19 2020, 6:13 AM
dvyukov accepted this revision.Feb 19 2020, 6:41 AM

LGTM. Thanks!
Do you have commit access? Or do you want me to merge this?

This revision is now accepted and ready to land.Feb 19 2020, 6:41 AM
dfava added a comment.Feb 19 2020, 6:52 AM

Please merge it for me. I don't have commit access. Thanks!