HerrCai0907 marked 2 inline comments as done. HerrCai0907 added inline comments.
================ Comment at: clang/lib/Sema/SemaDeclCXX.cpp:11196 if (D.isFunctionDefinition()) Diag(D.getIdentifierLoc(), diag::err_deduction_guide_defines_function); + return IsValid; ---------------- rsmith wrote: > Should we return that we had an error after this one too? if `D.isFunctionDefinition()`, we can still create valid deduction guide. I think keeping it can avoid lots of meaningless error later. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149516/new/ https://reviews.llvm.org/D149516 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits