hokein marked an inline comment as done.
hokein added inline comments.

================
Comment at: clangd/CodeComplete.cpp:661
+  if (CompletedName.SSInfo) {
+    // For a namespace scope specifier ("ns::"), we have disabled Sema from
+    // collecting completion results by setting IncludeNamespaceLevelDecls to
----------------
sammccall wrote:
> I don't think we need this - we plan to avoid overlap by splitting on current 
> file vs other files - i'll put together a patch to make sema do what we want.
> Until we've got that done, having some duplicate results is OK (this feature 
> is experimental still).
Thanks, that makes sense. Reverted it.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41668



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

Reply via email to