[PATCH] D94424: [clangd] Make AST-based signals available to runWithPreamble.

Utkarsh Saxena via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 13 06:30:35 PST 2021


usaxena95 added inline comments.


================
Comment at: clang-tools-extra/clangd/unittests/TUSchedulerTests.cpp:720
+  #include "foo.h"
+  namespace ns1 {
+  namespace ns2 {
----------------
adamcz wrote:
> Can you add anonymous namespaces as well?
Good point. Avoided capturing anonymous namespaces.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94424



More information about the cfe-commits mailing list