ioeric added a comment.

In D57047#1366265 <https://reviews.llvm.org/D57047#1366265>, @kadircet wrote:

> LGTM, but hokein might know better whether to push the default options or the 
> one in ClangdServerOpts


I'm going to land this now to unbreak build. Feel free to comment or do further 
refactoring if the default is not the best thing to pass here. Currently, this 
doesn't seem to matter as it's not used in CodeCompletion.


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57047/new/

https://reviews.llvm.org/D57047



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

Reply via email to