================ @@ -13435,16 +13435,18 @@ void Sema::checkNonTrivialCUnion(QualType QT, SourceLocation Loc, void Sema::AddInitializerToDecl(Decl *RealDecl, Expr *Init, bool DirectInit) { // If there is no declaration, there was an error parsing it. Just ignore // the initializer. - if (!RealDecl || RealDecl->isInvalidDecl()) { + if (!RealDecl) { CorrectDelayedTyposInExpr(Init, dyn_cast_or_null<VarDecl>(RealDecl)); return; } if (CXXMethodDecl *Method = dyn_cast<CXXMethodDecl>(RealDecl)) { ---------------- mizvekov wrote:
Minor nit: We can use separate init-statement since we adopted C++17; You call. ```suggestion if (auto *Method = dyn_cast<CXXMethodDecl>(RealDecl); !Method->isInvalidDecl()) { ``` https://github.com/llvm/llvm-project/pull/88645 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits