Raise impact of DefineMemberBasedOnUse to at least match RemoveInvalidCall.

Author: Greg Titus

lib/Sema/CSSimplify.cpp

@@ -11115,7 +11115,7 @@ ConstraintSystem::SolutionKind ConstraintSystem::simplifyMemberConstraint(
 
       auto instanceTy = baseObjTy->getMetatypeInstanceType();
 
-      auto impact = 2;
+      auto impact = 10;
       // Impact is higher if the base type is any function type
       // because function types can't have any members other than self
       if (instanceTy->is<AnyFunctionType>()) {

test/Constraints/diagnostics.swift

@@ -1572,3 +1572,19 @@ func testNilCoalescingOperatorRemoveFix() {
   if ("" // This is a comment
       ?? "").isEmpty {} // expected-warning {{left side of nil coalescing operator '??' has non-optional type 'String', so the right side is never used}} {{1572:9-1573:12=}}
 }
+
+// https://github.com/apple/swift/issues/74617
+struct Foo_74617 {
+  public var bar: Float {
+    return 123
+  }
+
+  public static func + (lhs: Self, rhs: Self) -> Self {
+    return Self()
+  }
+}
+func testAddMemberVsRemoveCell() {
+  let a = Foo_74617()
+  let b = Foo_74617()
+  let c = (a + b).bar() // expected-error {{cannot call value of non-function type 'Float'}} {{22-24=}}
+}