[PATCH] D69328: [clangd] Propogate context in TUScheduler::run

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 23 02:54:41 PDT 2019


ilya-biryukov added inline comments.


================
Comment at: clang-tools-extra/clangd/TUScheduler.cpp:924
+                        Action = std::move(Action)]() mutable { Action(); };
+  PreambleTasks->runAsync(Name, std::move(ActionWithCtx));
 }
----------------
Maybe inline `ActionWithCtx`? This should look fine as the last argument of the function call.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69328





More information about the cfe-commits mailing list