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

Reply via email to