arphaman added a comment.

In https://reviews.llvm.org/D49523#1169621, @jkorous wrote:

> Alex, I am just wondering if we shouldn't rather create another 
> implementation of GlobalCompilationDatabase interface (something like 
> InMemoryGlobalCompilationDatabase), add it to ClangdServer and use it as the 
> first place to be searched in ClangdServer::getCompileCommand(PathRef File). 
> What do you think?


We can certainly create another class, but I'd be against adding it into 
`ClangdServer`. It makes sense to have just one CDB in there. If we had our own 
server instead of ClangdLSP then this class would be really useful, but for now 
this solution is probably preferable.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D49523



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to