================
@@ -1621,6 +1621,9 @@ DeclarationFragmentsBuilder::getSubHeading(const 
NamedDecl *Decl) {
              cast<CXXMethodDecl>(Decl)->isOverloadedOperator()) {
     Fragments.append(Decl->getNameAsString(),
                      DeclarationFragments::FragmentKind::Identifier);
+  } else if (dyn_cast<TagDecl>(Decl) &&
----------------
QuietMisdreavus wrote:

Could this use `isa<TagDecl>` instead? The extraneous pointer cast is probably 
not a big deal in the broad scheme, but using `dyn_cast` as a boolean check 
just feels weird to me.

https://github.com/llvm/llvm-project/pull/110689
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to