[PATCH] D91122: [clangd] Call hierarchy (XRefs layer, incoming calls)

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 23 01:46:46 PST 2020


kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.

This looks great! Thanks a lot for bearing with me and doing all of this!



================
Comment at: clang-tools-extra/clangd/XRefs.cpp:1613
+    if (auto CHI = declToCallHierarchyItem(*Decl))
+      Result.emplace_back(*std::move(CHI));
+  }
----------------
please put the deref inside `std::move`. as move semantics of `Optional` are optional (no pun intended :P), i.e. depends on `LLVM_HAS_RVALUE_REFERENCE_THIS`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91122/new/

https://reviews.llvm.org/D91122



More information about the cfe-commits mailing list