[clang] [Clang][NFCI] Cleanup the fix for default function argument substitution (PR #104911)

[Younan Zhang via cfe-commits]

================
@@ -285,7 +286,8 @@ Response HandleFunction(Sema &SemaRef, const FunctionDecl *Function,
     // If this function was instantiated from a specialized member that is
     // a function template, we're done.
     assert(Function->getPrimaryTemplate() && "No function template?");
-    if (Function->getPrimaryTemplate()->isMemberSpecialization())
+    if (!ForDefaultArgumentSubstitution &&
+        Function->getPrimaryTemplate()->isMemberSpecialization())
----------------
zyn0217 wrote:

Nope, I applied that patch and the problem persists.
I stepped in the logic a little, and it seemed we had bailed out too early before the specifier substitution.

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