[PATCH] D73451: [clangd] Update the include mappings for std symbols.

2020-01-27 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet added a comment. In D73451#1841595 , @hokein wrote: > In D73451#1841510 , @kadircet wrote: > > > we seem to be missing some symbols now, like `polymorphic_allocator` > > `random_shuffle` `gets` etc. > > >

[PATCH] D73451: [clangd] Update the include mappings for std symbols.

2020-01-27 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62202 tests passed, 0 failed and 815 were skipped. {icon times-circle color=red} clang-tidy: fail. clang-tidy found 15 errors and 0 warnings

[PATCH] D73451: [clangd] Update the include mappings for std symbols.

2020-01-27 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. In D73451#1841510 , @kadircet wrote: > we seem to be missing some symbols now, like `polymorphic_allocator` > `random_shuffle` `gets` etc. yeap, two majoir reasons: - some symbols

[PATCH] D73451: [clangd] Update the include mappings for std symbols.

2020-01-27 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 240520. hokein added a comment. refine the code to avoid confusion. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73451/new/ https://reviews.llvm.org/D73451 Files: clang-tools-extra/clangd/CSymbolMap.inc cl

[PATCH] D73451: [clangd] Update the include mappings for std symbols.

2020-01-27 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet added a comment. we seem to be missing some symbols now, like `polymorphic_allocator` `random_shuffle` `gets` etc. and there seems to be a functional change to the parser. Are they intentional, if so why? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revie

[PATCH] D73451: [clangd] Update the include mappings for std symbols.

2020-01-27 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62202 tests passed, 0 failed and 815 were skipped. {icon times-circle color=red} clang-tidy: fail. clang-tidy found 15 errors and 0 warnings

[PATCH] D73451: [clangd] Update the include mappings for std symbols.

2020-01-27 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: kadircet. Herald added subscribers: usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Use the latest offline version of cppreference. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73451