[PATCH] D53019: [clangd] dump xrefs information in dexp tool.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 9 08:09:20 PDT 2018


sammccall added inline comments.


================
Comment at: clangd/index/dex/dexp/Dexp.cpp:65
+  clang::clangd::SymbolID SymID;
+  // We choose the first one if there are overloaded symbols.
+  Index->fuzzyFind(Request, [&](const Symbol &Sym) {
----------------
I don't think this is reasonable behavior. I'd suggest either of:
 - processing all results (this function would return a vector)
 - treat this as an error, the error message should include the ID of each symbol so we can rerun the command in an unambiguous way


================
Comment at: clangd/index/dex/dexp/Dexp.cpp:173
+      llvm::outs()
+          << "Missing required argument: please provide -id or -name.\n";
+      return;
----------------
nit: just "id" with no dash


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53019





More information about the cfe-commits mailing list