[clang-tools-extra] [clang-doc] Add a breadcrumb navigation bar (PR #173297)
Paul Kirth via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 6 10:04:39 PST 2026
================
@@ -697,10 +697,46 @@ static TemplateParamInfo convertTemplateArgToInfo(const clang::Decl *D,
return TemplateParamInfo(Str);
}
+// Check if the DeclKind is one for which we support contextual relationships.
+// There might be other ContextDecls, like blocks, that we currently don't
+// handle at all.
+static bool isSupportedContext(Decl::Kind DeclKind) {
+ switch (DeclKind) {
+ case Decl::Kind::Record:
+ case Decl::Kind::CXXRecord:
+ case Decl::Kind::ClassTemplateSpecialization:
+ case Decl::Kind::ClassTemplatePartialSpecialization:
+ case Decl::Kind::Namespace:
+ return true;
+ default:
+ return false;
+ }
+}
+
+template <typename T> static void findParent(Info &I, const T *D) {
+ assert(D && "Invalid Decl");
+
+ // Only walk up contexts if D is a record or namespace.
+ if (!isSupportedContext(D->getKind()))
+ return;
+
+ const DeclContext *ParentCtx = dyn_cast<DeclContext>(D)->getLexicalParent();
+ while (ParentCtx) {
+ if (isSupportedContext(ParentCtx->getDeclKind())) {
+ // Break when we reach the first record or namespace.
+ I.ParentUSR = getUSRForDecl(dyn_cast<Decl>(ParentCtx));
+ break;
+ }
+ ParentCtx = ParentCtx->getParent();
----------------
ilovepi wrote:
I cant recall if getLexicalParent() can return a `nullptr` or if that's structurally impossible. We can either add a check, or assert that its non-null.
https://github.com/llvm/llvm-project/pull/173297
More information about the cfe-commits
mailing list