[PATCH] D131385: [clangd] Support for standard type hierarchy
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 17 00:29:09 PDT 2022
kadircet marked 2 inline comments as done.
kadircet added inline comments.
================
Comment at: clang-tools-extra/clangd/XRefs.cpp:1713
+ fillSuperTypes(*ParentDecl, TUPath, *ParentSym, RPSet);
+ Item.data.parents->emplace_back(ParentSym->data);
+ Item.parents->emplace_back(std::move(*ParentSym));
----------------
usaxena95 wrote:
> nit: std::move.
the copy here is actually intentional. as we need to preserve `data` both in current element and also in its `parent`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D131385/new/
https://reviews.llvm.org/D131385
More information about the cfe-commits
mailing list