[PATCH] D54799: [clangd] textDocument/SymbolInfo method

2019-03-25 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Herald added a subscriber: jdoerfert. Herald added a project: LLVM. SymbolInfoTests.All takes > 20% of all of check-clang-tools time. Can the test be made faster? Also, since it's using a large-ish non-pod initializer clang is currently slow to compile it, see http://ll

[PATCH] D54799: [clangd] textDocument/SymbolInfo method

2019-03-25 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov added a comment. In D54799#1441409 , @thakis wrote: > SymbolInfoTests.All takes > 20% of all of check-clang-tools time. Can the > test be made faster? > > Also, since it's using a large-ish non-pod initializer clang is currently > slow to c

[PATCH] D54799: [clangd] textDocument/SymbolInfo method

2018-11-27 Thread Jan Korous via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL347674: [clangd][NFC] Move SymbolID to a separate file (authored by jkorous, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D54799?vs=175475&i

[PATCH] D54799: [clangd] textDocument/SymbolInfo method

2018-11-27 Thread Jan Korous via Phabricator via cfe-commits
jkorous added a comment. Going to split `SymbolID.h ` as a NFC commit per Alex's suggestion and land this. Thanks everyone! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54799/new/ https://reviews.llvm.org/D54799 ___ cfe-commits mailing li