erichkeane accepted this revision. erichkeane added inline comments.
================ Comment at: clang/lib/Sema/SemaDecl.cpp:176 + auto *FoundRD = dyn_cast<CXXRecordDecl>(TD); + if (DCK != DiagCtorKind::None && LookupRD && FoundRD && + FoundRD->isInjectedClassName() && ---------------- So I know our coding standard is a little awkward here, but I'd prefer curley braces here. I think the 'size' of the Diag line here qualifies as 'multiple lines'. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136533/new/ https://reviews.llvm.org/D136533 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
