kadircet added a comment.

Can you also add a test, preferably in clang/test/CodeCompletion/overrides.cpp ?



================
Comment at: clang/lib/Parse/ParseDecl.cpp:3269
 
-      if (getCurScope()->getFnParent() || getCurScope()->getBlockParent())
-        CCC = Sema::PCC_LocalDeclarationSpecifiers;
-      else if (TemplateInfo.Kind != ParsedTemplateInfo::NonTemplate)
+      if (TemplateInfo.Kind != ParsedTemplateInfo::NonTemplate)
         CCC = DSContext == DeclSpecContext::DSC_class ? 
Sema::PCC_MemberTemplate
----------------
i think the importance of ordering requires a comment here, maybe something 
like:
`Class context can appear inside a function/block, so prioritise that. `


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130363/new/

https://reviews.llvm.org/D130363

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to