[PATCH] D51163: [clangd] Check for include overlapping looks for only the line now.

2018-08-23 Thread Kadir Cetinkaya via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL340539: [clangd] Check for include overlapping looks for only the line now. (authored by kadircet, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM

[PATCH] D51163: [clangd] Check for include overlapping looks for only the line now.

2018-08-23 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D51163 ___ cfe-commits mailing list

[PATCH] D51163: [clangd] Check for include overlapping looks for only the line now.

2018-08-23 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet created this revision. kadircet added reviewers: ilya-biryukov, hokein, ioeric. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay. Currently we match an include only if we are inside filename, with this patch we will match whenever we are on the starting line of the