[PATCH] D41996: [clangd] Code completion uses Sema for NS-level things in the current file.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 12 09:22:45 PST 2018


hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.

nice, LGTM.

By `r322377`, I think you mean `r322371`.

We also need to set `CollectorOpts.IndexMainFiles` to `false` in  `FileIndex.cpp::indexAST` -- we don't need dynamic index to catch the symbols in current main file since sema does that for us now :)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41996





More information about the cfe-commits mailing list