ilya-biryukov added inline comments.
================
Comment at: clang-tools-extra/clangd/index/dex/dexp/CMakeLists.txt:10
+ Dexp.cpp
+ )
+
----------------
Should we indent closing parens to the opening one or keep at the start of the
line?
Let's pick one style and be consistent (the best option is being consistent
with the rest of LLVM/clangd)
================
Comment at: clang-tools-extra/clangd/index/dex/dexp/Dexp.cpp:46
+
+template <class Function> void reportTime(StringRef Name, Function F) {
+ const auto TimerStart = std::chrono::high_resolution_clock::now();
----------------
NIT: Now that we ignore the return value, we could even remove templates:
```
void reportTime(StringRef Name, llvm::function_ref<void()> F);
```
https://reviews.llvm.org/D51628
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
