Add description to shouldPatchPlaceholder0

Updated now. Thank you again for your patient review. :)

Address comments

Update doxygen comments

Address the comments. Revise parameters of getSignature to avoid passing CCR

Ping :)

Update

Address the comments

Move parsing logic to Record.cpp

Ping

Apply clang-format to StandardLibrary.cpp to fix pipeline failure

Thank you for your detailed explanation and sorry again for my dumb mistake before. :)

Revise

Rebase to main && Move tests to clang/Tooling

Sorry for the late update, I'm rebasing my branch now.

Revise symbols from N4100

In D143319#4115186, @kadircet wrote:

thanks a lot for the patch!

we're migrating IWYU related functionality in clangd to make use of include-cleaner library nowadays. so can you actually move the IWYU no_include pragma handling logic into https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/include-cleaner/lib/Record.cpp?
This won't have affect on clangd regarding the particular problem you're facing immediately, but we're going to merge the logic inside include-cleaner and clangd soon and such discrepancies should disappear then.

in the meanwhile regarding the particular problem you're facing for experimental symbols. sorry for shutting down D142836, but that actually made us re-evaluate the approach of hand-curated lists and we've decided to introduce such mappings. so if you can revamp D142836 and put TsStdSymbolMap.inc in https://github.com/llvm/llvm-project/tree/main/clang/lib/Tooling/Inclusions/Stdlib/StdTsSymbolMap.inc we can land it and make sure it work as intended inside clangd, even before we merge the IWYU pragma handling logic. Happy to do that myself if you don't feel like doing it, sorry for creating some confusion here :(

Ping

Fix test on windows

Update tests && Use Preprocessor.LookupFile

we're already having some efforts right now to improve our cppreference parsing and get to a more complete set of symbols (@VitaNuo for visibility, who drives these efforts)..

Format

Do not format inc file

Can anyone else help me to commit this PR? I mean we should land this patch ASAP, as many of PRs are now depending on llvm::ArrayRef without including ADT/ArrayRef.h but ADT/STLExtras.h, which would make it extremely harder to resolve this circular dependency later.

Fix issue

Ping again. Sorry I don't have commit access to main branch.

In D141352#4053069, @JDevlieghere wrote:

LGTM

Ping. Is this OK for trunk?

Add header llvm/ADT/Hashing.h, which is included by ArrayRef.h before. This header may be dependent by some TUs indirectly.

Pipeline failed due to missing include header llvm/ADT/Hashing.h

In file included from /var/lib/buildkite-agent/builds/llvm-project/clang-tools-extra/clangd/Config.cpp:9:
/var/lib/buildkite-agent/builds/llvm-project/clang-tools-extra/clangd/Config.h:158:18: error: no member named 'hash_combine' in namespace 'llvm'
    return llvm::hash_combine(Val.Kind, Val.Location, Val.MountPoint);