[PATCH] D108119: Wiring of standard library indexing into clangd.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 17 00:05:53 PDT 2021


sammccall added a comment.

(First just the bug that I suspect is blocking you from testing this, design feedback to come)



================
Comment at: clang-tools-extra/clangd/ClangdServer.cpp:184
+    }
+    AddIndex(SLIndex->get());
+  }
----------------
this pointer will dangle after the `if` block finishes (the local `auto` owns the index)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108119



More information about the cfe-commits mailing list