[PATCH] D39705: [clangd] Fix opening declarations located in non-preamble inclusion

2017-11-07 Thread Marc-Andre Laperle via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL317585: [clangd] Fix opening declarations located in non-preamble inclusion (authored by malaperle). Repository: rL LLVM https://reviews.llvm.org/D39705 Files: clang-tools-extra/trunk/clangd/ClangdU

[PATCH] D39705: [clangd] Fix opening declarations located in non-preamble inclusion

2017-11-07 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. LGTM Comment at: clangd/ClangdUnit.cpp:970 +SourceMgr.getFileEntryForID(SourceMgr.getFileID(LocStart))) { + StringRef FilePath = F->tryGetRealPathName(