[clang] [Clang] Implement CWG2918 'Consideration of constraints for address of overloaded function' (PR #127773)

[Erich Keane via cfe-commits]

================
@@ -13443,9 +13499,9 @@ Sema::resolveAddressOfSingleOverloadCandidate(Expr *E, DeclAccessPair &Pair) {
     return nullptr;
 
   if (Result) {
-    SmallVector<const Expr *, 1> ResultAC;
     // We skipped over some ambiguous declarations which might be ambiguous with
     // the selected result.
+    // FIXME: This for-loop is dead code. Remove it?
----------------
erichkeane wrote:

Hmm... perhaps this was supposed to be higher? in the code?  I'm concerned about it, but if no tests fail, then I think we have to go with it.  Just kill the loop.

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