[PATCH] D142836: [clangd] Add symbol mappings for `std::experimental::filesystem`

Younan Zhang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 29 23:13:10 PST 2023


zyounan added a comment.

In D142836#4089721 <https://reviews.llvm.org/D142836#4089721>, @kadircet wrote:

> And to follow up, I definitely see the problem you're facing and it's something we'd like to address as well, just not in the way you proposed.
>
> This falls under the bucket of "we might have symbols missing from our stdlib mappings and should try to compensate for that", e.g. we should probably try disabling include-insertion for such cases where we have a high confidence that the symbol is part of stdlib but we don't have an exact mapping in our symbol database to remedy the issue around inserting "wrong" includes.

Yes, this is another issue I want to address, but you've already pointed out that the parser is improving.

The cppref archive page is not always up to date. As of now, the official version is still dumped in 2019, that means we're likely missing (partial) symbols from C++20/23, e.g. `std::expected`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142836



More information about the cfe-commits mailing list