[clang] [Clang] diagnose deleted/default redeclaration of defined friend functions (PR #136717)

[Younan Zhang via cfe-commits]

================
@@ -142,6 +142,14 @@ NamedDecl *Parser::ParseCXXInlineMethodDef(
       SkipUntil(tok::semi);
     }
 
+    if (FunctionDecl *FD =
+            dyn_cast_if_present<FunctionDecl>(FnD->getPreviousDecl())) {
+      if (isa<CXXRecordDecl>(FD->getLexicalDeclContext()) ||
+          Actions.getDefaultedFunctionKind(FD).asComparison() ==
+              Sema::DefaultedComparisonKind::None)
----------------
zyn0217 wrote:

> if cases like comparisons or declarations outside of scope are not skipped, these diagnostics will be replaced with a general redefinition diagnostic message

Then how specifically would the patch improve with this “skipping”? Could you please elaborate on the difference?

https://github.com/llvm/llvm-project/pull/136717