[clang] [Clang] Avoid unnecessary overload resolution when building RecoveryExpr (PR #222296)

[Younan Zhang via cfe-commits]

================
@@ -14634,8 +14647,13 @@ static bool DiagnoseTwoPhaseLookup(
       R.suppressDiagnostics();
 
       OverloadCandidateSet Candidates(FnLoc, CSK);
-      SemaRef.AddOverloadedCallCandidates(R, ExplicitTemplateArgs, Args,
-                                          Candidates);
+      for (LookupResult::iterator I = R.begin(), E = R.end(); I != E; ++I) {
+        if (InvalidCandidates.contains(I->getAsFunction()))
----------------
zyn0217 wrote:

This is even better: https://llvm-compile-time-tracker.com/compare.php?from=678058af7da6a20593d8bb23076dc4773f9e55f1&to=e35e532d96a9e30acc9c2bd18878c94ca4b4d6e5&stat=instructions%3Au

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