[PATCH] D42517: [clangd] RFC: Pass Context implicitly using TLS.

2018-01-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 131564. sammccall added a comment. Rebase on the Span changes that landed as r323511 Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D42517 Files: clangd/ClangdLSPServer.cpp clangd/ClangdLSPServer.h clangd/ClangdServer.cpp clangd/Cl

[PATCH] D42517: [clangd] RFC: Pass Context implicitly using TLS.

2018-01-25 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov added a comment. LG. Let's move the APIs to callbacks and start passing Contexts everywhere. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D42517 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llv

[PATCH] D42517: [clangd] RFC: Pass Context implicitly using TLS.

2018-01-24 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: ilya-biryukov. Herald added subscribers: cfe-commits, ioeric, jkorous-apple, klimek. This is mostly to show what it would look like. I think it's pretty nice! The lit tests pass, but the unit tests don't build because I ripped out some