[PATCH] D67172: [clangd] Use pre-populated mappings for standard symbols

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 6 05:00:38 PDT 2019


hokein added a comment.

In D67172#1660469 <https://reviews.llvm.org/D67172#1660469>, @ilya-biryukov wrote:

> Ok, agreed. Adding this indirection layer is probably making things a bit too complex. I'll update the patch to clearly separate the std symbol mapping and canonical includes.
>
> In D67172#1660435 <https://reviews.llvm.org/D67172#1660435>, @hokein wrote:
>
> > And I'm not sure whether it is possible to remove the suffix mapping completely -- for STL symbols it is possible, but for other symbols like GNU C headers, I didn't find out an automatic way to build a lookup table.
>
>
> It's because we don't have anything similar to cppreference.com for them? Maybe we could try scrapping man pages ?


yes, cppreferences only provides symbols from C++/C standard libraries. Parsing man page is possible, but it requires some work (we have to write another tool to parse it).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67172





More information about the cfe-commits mailing list