Page MenuHomePhabricator

Introduce ThreadType enum
ClosedPublic

Authored by yuri on Feb 6 2019, 12:42 PM.

Details

Reviewers
dvyukov
Summary

Replace bool workerthread flag with ThreadType enum.
This change is preparation for fiber support.

Diff Detail

Event Timeline

yuri created this revision.Feb 6 2019, 12:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2019, 12:42 PM
Herald added subscribers: Restricted Project, llvm-commits, kubamracek. · View Herald Transcript
vitalybuka added inline comments.
lib/sanitizer_common/sanitizer_thread_registry.h
33

Can you expand GCD?

yuri updated this revision to Diff 185626.Feb 6 2019, 1:20 PM

More detailed comment

yuri marked 2 inline comments as done.Feb 6 2019, 1:21 PM
yuri added inline comments.
lib/sanitizer_common/sanitizer_thread_registry.h
33

Done

yuri marked an inline comment as done.Feb 6 2019, 1:22 PM
dvyukov accepted this revision.Feb 7 2019, 3:00 AM
This revision is now accepted and ready to land.Feb 7 2019, 3:00 AM

Committed in 353390
http://llvm.org/viewvc/llvm-project?view=revision&revision=353390

I fixed build of sanitizer_thread_registry_test.cc before committing.

dvyukov closed this revision.Feb 7 2019, 3:01 AM