[PATCH] D95450: [clangd] Respect ReferencesParams.context.includeDeclarations

2021-02-01 Thread Sam McCall via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. sammccall marked 2 inline comments as done. Closed by commit rGff4832dbff0c: [clangd] Respect ReferencesParams.context.includeDeclarations (authored by sammccall). Cha

[PATCH] D95450: [clangd] Respect ReferencesParams.context.includeDeclarations

2021-02-01 Thread Sam McCall via Phabricator via cfe-commits
sammccall added inline comments. Comment at: clang-tools-extra/clangd/XRefs.h:86 + enum ReferenceAttributes : unsigned { +Plain = 0, +Declaration = 1 << 0, kbobyrev wrote: > nit: `Plain` means neither `Declaration` nor `Definition`, right? Maybe > `None

[PATCH] D95450: [clangd] Respect ReferencesParams.context.includeDeclarations

2021-01-29 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev accepted this revision. kbobyrev added a comment. This revision is now accepted and ready to land. Thanks, LGTM with a couple of nits. Comment at: clang-tools-extra/clangd/XRefs.h:23 #include "clang/AST/Type.h" +#include "clang/Basic/BitmaskEnum.h" #include "clang/Fo

[PATCH] D95450: [clangd] Respect ReferencesParams.context.includeDeclarations

2021-01-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: kbobyrev. Herald added subscribers: usaxena95, kadircet, arphaman. sammccall requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. Unfortunately this treats overr