Move SwiftParser/Diagnostics to its own module

Just a bit of cleanup I wanted to do to keep the modules small and focused.

The idea is that SwiftParser just creates a SwiftSyntax tree and SwiftParserDiagnositcs produces nice error messages for such a tree.

Author: ahoppen

Package.swift

@@ -116,7 +116,7 @@ let package = Package(
     ),
     .target(
       name: "SwiftParser",
-      dependencies: ["SwiftBasicFormat", "SwiftDiagnostics", "SwiftSyntax"],
+      dependencies: ["SwiftDiagnostics", "SwiftSyntax"],
       exclude: [
         "CMakeLists.txt",
         "README.md",
@@ -125,6 +125,13 @@ let package = Package(
         "DeclarationAttribute.swift.gyb",
       ]
     ),
+    .target(
+      name: "SwiftParserDiagnostics",
+      dependencies: ["SwiftBasicFormat", "SwiftDiagnostics", "SwiftSyntax"],
+      exclude: [
+        "CMakeLists.txt",
+      ]
+    ),
     .target(
       name: "SwiftOperators",
       dependencies: ["SwiftSyntax", "SwiftParser", "SwiftDiagnostics"],
@@ -134,8 +141,8 @@ let package = Package(
     ),
     .target(
       name: "SwiftCompilerSupport",
-      dependencies: [
-        "SwiftSyntax", "SwiftParser", "SwiftDiagnostics", "SwiftOperators"],
+      dependencies: ["SwiftSyntax", "SwiftParser", "SwiftParserDiagnostics",
+                     "SwiftDiagnostics", "SwiftOperators"],
       exclude: [
         "CMakeLists.txt",
         "SwiftCompilerSupport.h"
@@ -155,7 +162,7 @@ let package = Package(
     ),
     .executableTarget(
       name: "swift-parser-cli",
-      dependencies: ["SwiftDiagnostics", "SwiftSyntax", "SwiftParser", "SwiftOperators", "_SwiftSyntaxMacros",
+      dependencies: ["SwiftDiagnostics", "SwiftSyntax", "SwiftParser", "SwiftParserDiagnostics", "SwiftOperators", "_SwiftSyntaxMacros",
                      .product(name: "ArgumentParser", package: "swift-argument-parser")]
     ),
     .testTarget(
@@ -191,6 +198,10 @@ let package = Package(
       dependencies: ["SwiftDiagnostics", "SwiftOperators", "SwiftParser",
                      "_SwiftSyntaxTestSupport", "SwiftSyntaxBuilder"]
     ),
+    .testTarget(
+      name: "SwiftParserDiagnosticsTest",
+      dependencies: ["SwiftDiagnostics", "SwiftParserDiagnostics"]
+    ),
     .testTarget(
       name: "SwiftOperatorsTest",
       dependencies: ["SwiftOperators", "_SwiftSyntaxTestSupport",

Sources/CMakeLists.txt

@@ -10,6 +10,7 @@ add_subdirectory(SwiftBasicFormat)
 add_subdirectory(SwiftSyntax)
 add_subdirectory(SwiftDiagnostics)
 add_subdirectory(SwiftParser)
+add_subdirectory(SwiftParserDiagnostics)
 add_subdirectory(SwiftOperators)
 add_subdirectory(SwiftSyntaxBuilder)
 add_subdirectory(_SwiftSyntaxMacros)

Sources/SwiftCompilerSupport/CMakeLists.txt

@@ -14,6 +14,7 @@ target_link_libraries(SwiftCompilerSupport PUBLIC
   SwiftSyntax
   SwiftDiagnostics
   SwiftParser
+  SwiftParserDiagnostics
   SwiftOperators
 )
 

Sources/SwiftCompilerSupport/ConsistencyCheck.swift

@@ -12,6 +12,7 @@
 
 import SwiftOperators
 @_spi(Testing) @_spi(RawSyntax) import SwiftParser
+import SwiftParserDiagnostics
 @_spi(RawSyntax) import SwiftSyntax
 
 extension Syntax {

Sources/SwiftParser/CMakeLists.txt

@@ -43,7 +43,6 @@ add_library(SwiftParser STATIC
   gyb_generated/TypeAttribute.swift)
 
 target_link_libraries(SwiftParser PUBLIC
-  SwiftBasicFormat
   SwiftSyntax
   SwiftDiagnostics)
 

Sources/SwiftParserDiagnostics/CMakeLists.txt

@@ -0,0 +1,34 @@
+# This source file is part of the Swift.org open source project
+#
+# Copyright (c) 2014 - 2022 Apple Inc. and the Swift project authors
+# Licensed under Apache License v2.0 with Runtime Library Exception
+#
+# See http://swift.org/LICENSE.txt for license information
+# See http://swift.org/CONTRIBUTORS.txt for Swift project authors
+
+add_library(SwiftParserDiagnostics STATIC
+  DiagnosticExtensions.swift
+  MissingNodesError.swift
+  ParserDiagnosticMessages.swift
+  ParseDiagnosticsGenerator.swift
+  PresenceUtils.swift
+  SyntaxExtensions.swift
+  Utils.swift)
+
+target_link_libraries(SwiftParserDiagnostics PUBLIC
+  SwiftBasicFormat
+  SwiftSyntax
+  SwiftDiagnostics)
+
+set_property(GLOBAL APPEND PROPERTY SWIFTSYNTAX_EXPORTS SwiftParserDiagnostics)
+
+# NOTE: workaround for CMake not setting up include flags yet
+set_target_properties(SwiftParserDiagnostics PROPERTIES
+  INTERFACE_INCLUDE_DIRECTORIES
+  "${CMAKE_Swift_MODULE_DIRECTORY} ${CMAKE_CURRENT_SOURCE_DIR}")
+
+install(TARGETS SwiftParserDiagnostics
+  EXPORT SwiftSyntaxTargets
+  ARCHIVE DESTINATION lib
+  LIBRARY DESTINATION lib
+  RUNTIME DESTINATION bin)

Sources/SwiftParser/Diagnostics/DiagnosticExtensions.swift renamed to Sources/SwiftParserDiagnostics/DiagnosticExtensions.swift

@@ -14,6 +14,7 @@ import SwiftDiagnostics
 import SwiftSyntax
 @_spi(Testing) import _SwiftSyntaxMacros
 import SwiftParser
+import SwiftParserDiagnostics
 import SwiftOperators
 import Foundation
 import ArgumentParser

Sources/SwiftParser/Diagnostics/MissingNodesError.swift renamed to Sources/SwiftParserDiagnostics/MissingNodesError.swift

@@ -1,6 +1,6 @@
 import XCTest
 import SwiftDiagnostics
-import SwiftParser
+import SwiftParserDiagnostics
 
 public class DiagnosticInfrastructureTests: XCTestCase {
   public func testDiagnosticID() throws {

Sources/SwiftParser/Diagnostics/ParseDiagnosticsGenerator.swift renamed to Sources/SwiftParserDiagnostics/ParseDiagnosticsGenerator.swift

@@ -1,6 +1,7 @@
 import XCTest
 @_spi(RawSyntax) import SwiftSyntax
 @_spi(Testing) @_spi(RawSyntax)  import SwiftParser
+import SwiftParserDiagnostics
 import SwiftDiagnostics
 import _SwiftSyntaxTestSupport
 

Sources/SwiftParser/Diagnostics/ParserDiagnosticMessages.swift renamed to Sources/SwiftParserDiagnostics/ParserDiagnosticMessages.swift

@@ -3,6 +3,7 @@ import Dispatch
 import XCTest
 import SwiftSyntax
 import SwiftParser
+import SwiftParserDiagnostics
 
 public class ParserTests: XCTestCase {
   /// Run a single parse test.