[PATCH] D63714: [clangd] Cleanup the duplicated getTokenRange.

2019-06-25 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL364280: [clangd] Cleanup the duplicated getTokenRange. (authored by hokein, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://review

[PATCH] D63714: [clangd] Cleanup the duplicated getTokenRange.

2019-06-25 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 206380. hokein marked an inline comment as done. hokein added a comment. Address review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63714/new/ https://reviews.llvm.org/D63714 Files: clang-tools-ex

[PATCH] D63714: [clangd] Cleanup the duplicated getTokenRange.

2019-06-24 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/XRefs.cpp:926 for (const auto &Ref : MainFileRefs) { Location Result; +if (auto Range = nit: move de

[PATCH] D63714: [clangd] Cleanup the duplicated getTokenRange.

2019-06-24 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: kadircet. Herald added subscribers: arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Also lift it to SourceCode.h, so that it can be used in other places (semantic code highlighting). Repository: rG LLVM Github Mon