[clang] [Clang][Sema] Diagnose function/variable templates that shadow their own template parameters (PR #78274)

[Erich Keane via cfe-commits]

================
@@ -6070,7 +6070,7 @@ static TypeSourceInfo *GetFullTypeForDeclarator(TypeProcessingState &state,
 ///
 /// The result of this call will never be null, but the associated
 /// type may be a null type if there's an unrecoverable error.
-TypeSourceInfo *Sema::GetTypeForDeclarator(Declarator &D, Scope *S) {
+TypeSourceInfo *Sema::GetTypeForDeclarator(Declarator &D) {
----------------
erichkeane wrote:

How does removing this, which appears to not be used, fix the issue?  Or is it only the moved decl search in HandleDeclarator that does that?

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