[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