Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix test hanging in MultiQueueWaiter destructor && allow it to run with clang >= 6.0.0 #680

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

xerrni
Copy link
Contributor

@xerrni xerrni commented Jul 26, 2020

First commit fixes ccls hanging in MultiQueueWaiter destructor during test execution.
Second one enables tests execution when clang version >= 6.0.0.

MaskRay and others added 5 commits July 4, 2020 10:05
Add WorkDoneProgress to represent WorkDoneProgressBegin/WorkDoneProgressReport/WorkDoneProgressEnd.
std::condition_variable_any destructor is only safe to invoke if all
threads have been notified.

Reference:
https://en.cppreference.com/w/cpp/thread/condition_variable_any/%7Econdition_variable_any

Signed-off-by: Ernest Borowski <[email protected]>
@xerrni xerrni changed the title Fix test hanging in MultiQueueWaiter destructor && allow it to run on clang >= 6.0.0 Fix test hanging in MultiQueueWaiter destructor && allow it to run with clang >= 6.0.0 Jul 26, 2020
@MaskRay MaskRay force-pushed the master branch 3 times, most recently from ee29996 to feb153a Compare December 20, 2020 03:01
@MaskRay MaskRay force-pushed the master branch 9 times, most recently from 94ba2b3 to c018bce Compare May 9, 2021 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants