ilya-biryukov added inline comments.

================
Comment at: clangd/ClangdServer.h:121-122
+public:
+  /// Returns the number of threads to use when shouldRunsynchronously() is
+  /// false. Must not be called if shouldRunsynchronously() is true.
+  unsigned getThreadsCount();
----------------
ilya-biryukov wrote:
> klimek wrote:
> > Why not: 1 -> run synchronously, > 1, run in parallel?
> Currently 1 means: start 1 worker thread to run async operations (that thread 
> is separate from the main thread).
> This makes sense for clangd, as if you do that, you still get code completion 
> that doesn't wait for diagnostics to finish.
> On the other hand, it's useful to have `-run-synchronously` for some tests.
As discussed, replaced with `unsigned AsyncThreadsCount`. Zero means run 
synchronously.


https://reviews.llvm.org/D36261



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

Reply via email to