[PATCH] D142467: [Tooling] Add stdlib::Symbol::all() and stdlib::Symbol::qualified_name()

[Haojian Wu via Phabricator via cfe-commits]

hokein accepted this revision.
hokein added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/lib/Tooling/Inclusions/Stdlib/StandardLibrary.cpp:18
 
 static llvm::StringRef *HeaderNames;
+static struct SymbolName {
----------------
nit: we group five symbols together, it is a bit hard to follow.

Can you move the HeaderNames just before the `HeaderIDs` and mention the value of the `HeaderIDs`  is the index of `HeaderNames`?


================
Comment at: clang/lib/Tooling/Inclusions/Stdlib/StandardLibrary.cpp:108
+  std::vector<Symbol> Result;
+  Result.reserve(HeaderIDs->size());
+  for (unsigned I = 0, E = SymbolCount; I < E; ++I)
----------------
should be `SymbolCount`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142467