Merge branch 'main' into merge-main-6.2-2025-04-10

Author: ahoppen

.github/workflows/pull_request.yml

@@ -4,12 +4,14 @@ on:
   pull_request:
     types: [opened, reopened, synchronize]
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   tests:
     name: Test
     uses: swiftlang/github-workflows/.github/workflows/swift_package_test.yml@main
-    with:
-      linux_exclude_swift_versions: "[{\"swift_version\": \"5.8\"}]"
   soundness:
     name: Soundness
     uses: swiftlang/github-workflows/.github/workflows/soundness.yml@main

BUILD.bazel

@@ -219,6 +219,9 @@ swift_syntax_library(
         ":SwiftSyntax509",
         ":SwiftSyntax510",
         ":SwiftSyntax600",
+        ":SwiftSyntax601",
+        ":SwiftSyntax602",
+        ":SwiftSyntax603",
         ":_SwiftSyntaxCShims",
     ],
 )
@@ -251,6 +254,20 @@ swift_syntax_library(
     ],
 )
 
+swift_syntax_library(
+    name = "SwiftSyntax602",
+    srcs = glob(["Sources/VersionMarkerModules/SwiftSyntax602/**/*.swift"]),
+    deps = [
+    ],
+)
+
+swift_syntax_library(
+    name = "SwiftSyntax603",
+    srcs = glob(["Sources/VersionMarkerModules/SwiftSyntax603/**/*.swift"]),
+    deps = [
+    ],
+)
+
 swift_syntax_library(
     name = "SwiftSyntaxBuilder",
     deps = [
@@ -376,6 +393,7 @@ cc_library(
 
 cc_library(
     name = "_SwiftSyntaxCShims",
+    srcs = glob(["Sources/_SwiftSyntaxCShims/*.c"]),
     hdrs = glob(["Sources/_SwiftSyntaxCShims/include/*.h"]),
     includes = ["Sources/_SwiftSyntaxCShims/include"],
     tags = ["swift_module=_SwiftSyntaxCShims"],

Package.swift

@@ -210,7 +210,10 @@ let package = Package(
 
     .target(
       name: "SwiftSyntax",
-      dependencies: ["_SwiftSyntaxCShims", "SwiftSyntax509", "SwiftSyntax510", "SwiftSyntax600"],
+      dependencies: [
+        "_SwiftSyntaxCShims", "SwiftSyntax509", "SwiftSyntax510", "SwiftSyntax600", "SwiftSyntax601", "SwiftSyntax602",
+        "SwiftSyntax603",
+      ],
       exclude: ["CMakeLists.txt"],
       swiftSettings: swiftSyntaxSwiftSettings
     ),
@@ -248,6 +251,11 @@ let package = Package(
       path: "Sources/VersionMarkerModules/SwiftSyntax602"
     ),
 
+    .target(
+      name: "SwiftSyntax603",
+      path: "Sources/VersionMarkerModules/SwiftSyntax603"
+    ),
+
     // MARK: SwiftSyntaxBuilder
 
     .target(

Sources/SwiftDiagnostics/DiagnosticsFormatter.swift

@@ -293,10 +293,13 @@ public struct DiagnosticsFormatter {
         )
       )
 
-      // If the line did not end with \n (e.g. the last line), append it manually
-      if annotatedSource.last != "\n" {
-        annotatedSource.append("\n")
+      // Remove any trailing newline and replace it; this may seem
+      // counterintuitive, but if we're running within CMake and we let a
+      // '\r\n' through, CMake will turn that into *two* newlines.
+      if let last = annotatedSource.last, last.isNewline {
+        annotatedSource.removeLast()
       }
+      annotatedSource.append("\n")
 
       let columnsWithDiagnostics = Set(
         annotatedLine.diagnostics.map {
@@ -331,8 +334,13 @@ public struct DiagnosticsFormatter {
       }
 
       // Add suffix text.
-      annotatedSource.append(annotatedLine.suffixText)
-      if annotatedSource.last != "\n" {
+      if !annotatedLine.suffixText.isEmpty {
+        annotatedSource.append(annotatedLine.suffixText)
+
+        // See above for an explanation of why we do this
+        if let last = annotatedSource.last, last.isNewline {
+          annotatedSource.removeLast()
+        }
         annotatedSource.append("\n")
       }
     }

Sources/SwiftOperators/OperatorTable+Folding.swift

@@ -141,7 +141,6 @@ extension OperatorTable {
     if let unsafeExpr = lhs.as(UnsafeExprSyntax.self) {
       return ExprSyntax(
         UnsafeExprSyntax(
-          leadingTrivia: unsafeExpr.leadingTrivia,
           unsafeExpr.unexpectedBeforeUnsafeKeyword,
           unsafeKeyword: unsafeExpr.unsafeKeyword,
           unsafeExpr.unexpectedBetweenUnsafeKeywordAndExpression,
@@ -150,8 +149,7 @@ extension OperatorTable {
             op: op,
             rhs: rhs
           ),
-          unsafeExpr.unexpectedAfterExpression,
-          trailingTrivia: unsafeExpr.trailingTrivia
+          unsafeExpr.unexpectedAfterExpression
         )
       )
     }

Sources/VersionMarkerModules/CMakeLists.txt

@@ -11,6 +11,12 @@
 add_library(${SWIFTSYNTAX_TARGET_NAMESPACE}SwiftSyntax509 STATIC
   SwiftSyntax509/Empty.swift)
 add_library(${SWIFTSYNTAX_TARGET_NAMESPACE}SwiftSyntax510 STATIC
-  SwiftSyntax509/Empty.swift)
+  SwiftSyntax510/Empty.swift)
 add_library(${SWIFTSYNTAX_TARGET_NAMESPACE}SwiftSyntax600 STATIC
-  SwiftSyntax509/Empty.swift)
+  SwiftSyntax600/Empty.swift)
+add_library(${SWIFTSYNTAX_TARGET_NAMESPACE}SwiftSyntax601 STATIC
+  SwiftSyntax601/Empty.swift)
+add_library(${SWIFTSYNTAX_TARGET_NAMESPACE}SwiftSyntax602 STATIC
+  SwiftSyntax602/Empty.swift)
+add_library(${SWIFTSYNTAX_TARGET_NAMESPACE}SwiftSyntax603 STATIC
+  SwiftSyntax603/Empty.swift)

Sources/VersionMarkerModules/SwiftSyntax603/Empty.swift

@@ -0,0 +1,15 @@
+//===----------------------------------------------------------------------===//
+//
+// This source file is part of the Swift.org open source project
+//
+// Copyright (c) 2014 - 2025 Apple Inc. and the Swift project authors
+// Licensed under Apache License v2.0 with Runtime Library Exception
+//
+// See https://swift.org/LICENSE.txt for license information
+// See https://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
+//
+//===----------------------------------------------------------------------===//
+
+// The SwiftSyntax603 module is intentionally empty.
+// It serves as an indicator which version of swift-syntax a package is building against.
+// See the 'Macro Versioning.md' document for more details.

Tests/SwiftOperatorsTest/OperatorTableTests.swift

@@ -435,4 +435,10 @@ class OperatorPrecedenceTests: XCTestCase {
     }
 
   }
+
+  func testTriviaAroundUnsafeExpr() throws {
+    let original = ExprSyntax("/*leading*/ unsafe a /*trailing*/ + b")
+    let folded = try OperatorTable.standardOperators.foldAll(original)
+    XCTAssertEqual(original.description, folded.description)
+  }
 }