[PATCH] D55185: [Clangd] Index main-file symbols (bug 39761)

[Sam McCall via Phabricator via cfe-commits]

sammccall updated this revision to Diff 181510.
sammccall added a comment.
Herald added a subscriber: cfe-commits.

add comment about subtleties around canonical declarations and main-file check
invert sense of Symbol flag (FileLocal -> VisibleOutsideFile) so Merge does the right thing
test: add case for forward declarations (I think we do the wrong thing, fix later)
test: replace Matcher(true)/Matcher(false) with Matcher() and Not(Matcher())


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D55185

Files:
  clangd/index/Index.h
  clangd/index/SymbolCollector.cpp
  clangd/index/SymbolCollector.h
  unittests/clangd/BackgroundIndexTests.cpp
  unittests/clangd/FindSymbolsTests.cpp
  unittests/clangd/SymbolCollectorTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55185.181510.patch
Type: text/x-patch
Size: 16985 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190114/ccdfd68b/attachment-0001.bin>