[PATCH] D74125: [clangd] Set "spelled" flag for constructor references.

2020-02-06 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGca9fd22adb5a: [clangd] Set "spelled" flag for constructor references. (authored by hokein). Changed prior to commit: https://reviews.llvm.org/D74125?vs=242890&id=242911#toc Repository: rG LLVM Github

[PATCH] D74125: [clangd] Set "spelled" flag for constructor references.

2020-02-06 Thread Haojian Wu via Phabricator via cfe-commits
hokein marked 3 inline comments as done. hokein added inline comments. Comment at: clang-tools-extra/clangd/index/SymbolCollector.cpp:597 +bool IsTargetKind = NameKind == DeclarationName::Identifier || + NameKind == DeclarationName::CXXConstructorName; +b

[PATCH] D74125: [clangd] Set "spelled" flag for constructor references.

2020-02-06 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev accepted this revision. kbobyrev added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/index/SymbolCollector.cpp:595 DeclarationName Name = DeclAndRef.first->getDeclName(); -bool Spelled = Identifier

[PATCH] D74125: [clangd] Set "spelled" flag for constructor references.

2020-02-06 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet added inline comments. Comment at: clang-tools-extra/clangd/index/SymbolCollector.cpp:597 +bool IsTargetKind = NameKind == DeclarationName::Identifier || + NameKind == DeclarationName::CXXConstructorName; +bool Spelled = IdentifierToken && IsTarg

[PATCH] D74125: [clangd] Set "spelled" flag for constructor references.

2020-02-06 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 242890. hokein added a comment. update. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74125/new/ https://reviews.llvm.org/D74125 Files: clang-tools-extra/clangd/index/SymbolCollector.cpp clang-tools-extra/c

[PATCH] D74125: [clangd] Set "spelled" flag for constructor references.

2020-02-06 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: kbobyrev. Herald added subscribers: usaxena95, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. DeclarationName for cxx constructor is special, it is not an identifier. thus the "Spelled" flag are not set fo