This is an archive of the discontinued LLVM Phabricator instance.

Restored r303067 and fixed failing test.
ClosedPublic

Authored by ilya-biryukov on May 16 2017, 2:46 AM.

Details

Summary

This commit restores r303067(reverted by r303094) and fixes the 'formatting.test'
failure.
The failure is due to destructors of ClangdLSPServer's fields(FixItsMap and
FixItsMutex) being called before destructor of Server. It led to the worker
thread calling consumeDiagnostics after FixItsMutex and FixItsMap
destructors were called.
Also, clangd is now run with '-run-synchronously' flag in 'formatting.test'.

Event Timeline

ilya-biryukov created this revision.May 16 2017, 2:46 AM
krasimir accepted this revision.May 16 2017, 2:49 AM
This revision is now accepted and ready to land.May 16 2017, 2:49 AM
ilya-biryukov closed this revision.May 16 2017, 2:52 AM
clangd/GlobalCompilationDatabase.h