Add missing colon in _SwiftifyImport function call (#78316)

Author: hnrklssn

lib/Macros/Sources/SwiftMacros/SwiftifyImportMacro.swift

@@ -235,8 +235,8 @@ struct FunctionCallBuilder: BoundsCheckedThunkBuilder {
         return pointerArgs[i] ?? ExprSyntax(declref)
       }
     let labels: [TokenSyntax?] = base.signature.parameterClause.parameters.map { param in
-      let firstName = param.firstName
-      if firstName.trimmed.text == "_" {
+      let firstName = param.firstName.trimmed
+      if firstName.text == "_" {
         return nil
       }
       return firstName
@@ -247,7 +247,8 @@ struct FunctionCallBuilder: BoundsCheckedThunkBuilder {
       if i < args.count - 1 {
         comma = .commaToken()
       }
-      return LabeledExprSyntax(label: label, expression: arg, trailingComma: comma)
+      let colon: TokenSyntax? = label != nil ? .colonToken() : nil
+      return LabeledExprSyntax(label: label, colon: colon, expression: arg, trailingComma: comma)
     }
     return ExprSyntax(
       FunctionCallExprSyntax(

test/Macros/SwiftifyImport/CountedBy/NamedParams.swift

@@ -0,0 +1,58 @@
+// REQUIRES: swift_swift_parser
+
+// RUN: %target-swift-frontend %s -swift-version 5 -module-name main -disable-availability-checking -typecheck -plugin-path %swift-plugin-dir -dump-macro-expansions 2>&1 | %FileCheck --match-full-lines %s
+
+@_SwiftifyImport(.countedBy(pointer: 1, count: "len"))
+func ptrNamed(ptr: UnsafePointer<CInt>, _ len: CInt) {
+}
+
+@_SwiftifyImport(.countedBy(pointer: 1, count: "len"))
+func ptrNamedOther(buf ptr: UnsafePointer<CInt>, _ len: CInt) {
+}
+
+@_SwiftifyImport(.countedBy(pointer: 1, count: "len"))
+func lenNamed(_ ptr: UnsafePointer<CInt>, len: CInt) {
+}
+
+@_SwiftifyImport(.countedBy(pointer: 1, count: "len"))
+func lenNamedOther(_ ptr: UnsafePointer<CInt>, count len: CInt) {
+}
+
+@_SwiftifyImport(.countedBy(pointer: 1, count: "len"))
+func allNamed(ptr: UnsafePointer<CInt>, len: CInt) {
+}
+
+@_SwiftifyImport(.countedBy(pointer: 1, count: "len"))
+func allNamedOther(buf ptr: UnsafePointer<CInt>, count len: CInt) {
+}
+
+// CHECK: @_alwaysEmitIntoClient
+// CHECK-NEXT: func ptrNamed(ptr: UnsafeBufferPointer<CInt>) {
+// CHECK-NEXT:     return ptrNamed(ptr: ptr.baseAddress!, CInt(exactly: ptr.count)!)
+// CHECK-NEXT: }
+
+// CHECK: @_alwaysEmitIntoClient
+// CHECK-NEXT: func ptrNamedOther(buf ptr: UnsafeBufferPointer<CInt>) {
+// CHECK-NEXT:     return ptrNamedOther(buf: ptr.baseAddress!, CInt(exactly: ptr.count)!)
+// CHECK-NEXT: }
+
+// CHECK: @_alwaysEmitIntoClient
+// CHECK-NEXT: func lenNamed(_ ptr: UnsafeBufferPointer<CInt>) {
+// CHECK-NEXT:     return lenNamed(ptr.baseAddress!, len: CInt(exactly: ptr.count)!)
+// CHECK-NEXT: }
+
+// CHECK: @_alwaysEmitIntoClient
+// CHECK-NEXT: func lenNamedOther(_ ptr: UnsafeBufferPointer<CInt>) {
+// CHECK-NEXT:     return lenNamedOther(ptr.baseAddress!, count: CInt(exactly: ptr.count)!)
+// CHECK-NEXT: }
+
+// CHECK: @_alwaysEmitIntoClient
+// CHECK-NEXT: func allNamed(ptr: UnsafeBufferPointer<CInt>) {
+// CHECK-NEXT:     return allNamed(ptr: ptr.baseAddress!, len: CInt(exactly: ptr.count)!)
+// CHECK-NEXT: }
+
+// CHECK: @_alwaysEmitIntoClient
+// CHECK-NEXT: func allNamedOther(buf ptr: UnsafeBufferPointer<CInt>) {
+// CHECK-NEXT:     return allNamedOther(buf: ptr.baseAddress!, count: CInt(exactly: ptr.count)!)
+// CHECK-NEXT: }
+