Address review comments.

Author: comintern

Rubberduck.Parsing/VBA/DeclarationCaching/DeclarationFinder.cs

@@ -541,21 +541,26 @@ public ParameterDeclaration FindParameterFromArgument(VBAParser.ArgumentExpressi
                 return null;
             }
 
-            var callStmt = argExpression?.GetAncestor<VBAParser.CallStmtContext>();
-            var procedureName = callStmt?.GetDescendent<VBAParser.LExpressionContext>()
-                                         .GetDescendents<VBAParser.IdentifierContext>()
-                                         .LastOrDefault()?.GetText();
-            if (procedureName == null)
+            var callStmt = argExpression.GetAncestor<VBAParser.CallStmtContext>();
+
+            var identifier = callStmt?
+                .GetDescendent<VBAParser.LExpressionContext>()
+                .GetDescendents<VBAParser.IdentifierContext>()
+                .LastOrDefault();
+
+            if (identifier == null)
             {
                 // if we don't know what we're calling, we can't dig any further
                 return null;
             }
 
-            var procedure = MatchName(procedureName)
-                .SingleOrDefault(p =>
-                    (!p.DeclarationType.HasFlag(DeclarationType.Property) ||
-                     p.DeclarationType.HasFlag(DeclarationType.PropertyGet)) &&
-                    p.References.Any(r => callStmt == r.Context.GetAncestor<VBAParser.CallStmtContext>()));
+            var selection = new QualifiedSelection(enclosingProcedure.QualifiedModuleName, identifier.GetSelection());
+            if (!_referencesBySelection.TryGetValue(selection, out var matches))
+            {
+                return null;
+            }
+
+            var procedure = matches.SingleOrDefault()?.Declaration;
             if (procedure?.ParentScopeDeclaration is ClassModuleDeclaration)
             {
                 // we can't know that the member is on the class' default interface