commit 217af05bd106347304676e169f97d8b6a1ebc3ca
author Ilya Biryukov <ibiryukov@google.com> Thu Jul 18 15:21:34 2019 +0000
committer Ilya Biryukov <ibiryukov@google.com> Thu Jul 18 15:21:34 2019 +0000
tree 1cdaf35d1c94455ff73a00b6c4c918e91d6c6df4
parent 9345ddca9eaab5d19f9b5c382c709ca12df5484b

[ASTUnit] Fix a regression in cached completions

Summary:
After r345152 cached completions started adding namespaces after
nested name specifiers, e.g. in `some_name::^`

The CCC_Symbol indicates the completed item cannot be a namespace (it is
described as being "a type, a function or a variable" in the comments).

Therefore, 'nested specifier' completions should only be added from cache
when the context is CCC_SymbolOrNewName (which roughly seems to indicate
that a nested name specifier is allowed).

Fixes https://bugs.llvm.org/show_bug.cgi?id=42646

Reviewers: kadircet, sammccall

Reviewed By: kadircet, sammccall

Subscribers: arphaman, nik, sammccall, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D64918

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@366448 91177308-0d34-0410-b5e6-96231b3b80d8