[PATCH] D73109: [clang][index] Index the injected class name types.

2020-01-22 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62058 tests passed, 0 failed and 784 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73109: [clang][index] Index the injected class name types.

2020-01-22 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGa133cbaad5b2: [clang][index] Index the injected class name types. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73109/new/ https://rev

[PATCH] D73109: [clang][index] Index the injected class name types.

2020-01-22 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. In D73109#1831399 , @kadircet wrote: > Have you checked if this class is missing any other typelocs? I haven't checked it in details, but the injected class name is an obvious missing case. Repository: rG LLVM Github Monorepo

[PATCH] D73109: [clang][index] Index the injected class name types.

2020-01-22 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 239504. hokein added a comment. clang-format the code. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73109/new/ https://reviews.llvm.org/D73109 Files: clang/lib/Index/IndexTypeSourceInfo.cpp clang/unittests

[PATCH] D73109: [clang][index] Index the injected class name types.

2020-01-21 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. Thanks this LG (please apply clang-format fixes though) Have you checked if this class is missing any other typelocs? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION ht

[PATCH] D73109: [clang][index] Index the injected class name types.

2020-01-21 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62058 tests passed, 0 failed and 784 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by runnin

[PATCH] D73109: [clang][index] Index the injected class name types.

2020-01-21 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: kadircet. Herald added subscribers: usaxena95, arphaman, ilya-biryukov. Herald added a project: clang. Currently, we (clangd) are missing this kind of references. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73109 Files: