This is an archive of the discontinued LLVM Phabricator instance.

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

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



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!