================ @@ -1398,11 +1398,16 @@ void Sema::ActOnEndOfTranslationUnit() { if (FD->getDescribedFunctionTemplate()) Diag(DiagD->getLocation(), diag::warn_unused_template) << /*function=*/0 << DiagD << DiagRange; - else - Diag(DiagD->getLocation(), isa<CXXMethodDecl>(DiagD) - ? diag::warn_unused_member_function - : diag::warn_unused_function) - << DiagD << DiagRange; + else { + if (isa<CXXMethodDecl>(DiagD)) + Diag(DiagD->getLocation(), diag::warn_unused_member_function) + << (!isa<CXXConstructorDecl>(DiagD) ? /*member function=*/0 + : /*constructor=*/1) + << DiagD << DiagRange; + else + Diag(DiagD->getLocation(), diag::warn_unused_function) + << DiagD << DiagRange; + } ---------------- cor3ntin wrote:
```suggestion else { Diag(DiagD->getLocation(), isa<CXXMethodDecl>(DiagD) ? diag::warn_unused_member_function : diag::warn_unused_function) << DiagD << (!isa<CXXConstructorDecl>(DiagD) ? /*member function=*/0 : /*constructor=*/1 << DiagRange; } ``` I think you can simplify like that. And then change ` warn_unused_member_function : Warning<"unused %select{member function|constructor}1 %0"` https://github.com/llvm/llvm-project/pull/84515 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits