Merge 'AtomicBool' into _SwiftSyntaxCShims

Author: rintaro

Package.swift

@@ -34,10 +34,6 @@ let package = Package(
       name: "_SwiftSyntaxCShims"
     ),
 
-    .target(
-      name: "_AtomicBool"
-    ),
-
     .target(
       name: "_InstructionCounter"
     ),
@@ -131,7 +127,7 @@ let package = Package(
 
     .target(
       name: "SwiftSyntax",
-      dependencies: ["_AtomicBool", "SwiftSyntax509", "SwiftSyntax510", "SwiftSyntax600"],
+      dependencies: ["_SwiftSyntaxCShims", "SwiftSyntax509", "SwiftSyntax510", "SwiftSyntax600"],
       exclude: ["CMakeLists.txt"],
       swiftSettings: swiftSyntaxSwiftSettings
     ),

Sources/SwiftCompilerPlugin/CompilerPlugin.swift

@@ -131,35 +131,31 @@ extension CompilerPlugin {
   /// Main entry point of the plugin — sets up a communication channel with
   /// the plugin host and runs the main message loop.
   public static func main() throws {
-    let stdin = _ss_stdin()
-    let stdout = _ss_stdout()
-    let stderr = _ss_stderr()
-
     // Duplicate the `stdin` file descriptor, which we will then use for
     // receiving messages from the plugin host.
-    let inputFD = dup(fileno(stdin))
+    let inputFD = dup(fileno(_stdin))
     guard inputFD >= 0 else {
-      internalError("Could not duplicate `stdin`: \(describe(errno: _ss_errno())).")
+      internalError("Could not duplicate `stdin`: \(describe(errno: _errno)).")
     }
 
     // Having duplicated the original standard-input descriptor, we close
     // `stdin` so that attempts by the plugin to read console input (which
     // are usually a mistake) return errors instead of blocking.
-    guard close(fileno(stdin)) >= 0 else {
-      internalError("Could not close `stdin`: \(describe(errno: _ss_errno())).")
+    guard close(fileno(_stdin)) >= 0 else {
+      internalError("Could not close `stdin`: \(describe(errno: _errno)).")
     }
 
     // Duplicate the `stdout` file descriptor, which we will then use for
     // sending messages to the plugin host.
-    let outputFD = dup(fileno(stdout))
+    let outputFD = dup(fileno(_stdout))
     guard outputFD >= 0 else {
-      internalError("Could not dup `stdout`: \(describe(errno: _ss_errno())).")
+      internalError("Could not dup `stdout`: \(describe(errno: _errno)).")
     }
 
     // Having duplicated the original standard-output descriptor, redirect
     // `stdout` to `stderr` so that all free-form text output goes there.
-    guard dup2(fileno(stderr), fileno(stdout)) >= 0 else {
-      internalError("Could not dup2 `stdout` to `stderr`: \(describe(errno: _ss_errno())).")
+    guard dup2(fileno(_stderr), fileno(_stdout)) >= 0 else {
+      internalError("Could not dup2 `stdout` to `stderr`: \(describe(errno: _errno)).")
     }
 
     #if canImport(ucrt)
@@ -189,7 +185,7 @@ extension CompilerPlugin {
 
   // Private function to report internal errors and then exit.
   fileprivate static func internalError(_ message: String) -> Never {
-    fputs("Internal Error: \(message)\n", _ss_stderr())
+    fputs("Internal Error: \(message)\n", _stderr)
     exit(1)
   }
 }
@@ -240,7 +236,7 @@ private func _write(_ fd: CInt, contentsOf buffer: UnsafeRawBufferPointer) throw
   let endPtr = ptr.advanced(by: buffer.count)
   while ptr != endPtr {
     switch write(fd, ptr, numericCast(endPtr - ptr)) {
-    case -1: throw IOError.writeFailed(errno: _ss_errno())
+    case -1: throw IOError.writeFailed(errno: _errno)
     case 0: throw IOError.writeFailed(errno: 0) /* unreachable */
     case let n: ptr += Int(n)
     }
@@ -255,7 +251,7 @@ private func _read(_ fd: CInt, into buffer: UnsafeMutableRawBufferPointer) throw
   let endPtr = ptr.advanced(by: buffer.count)
   while ptr != endPtr {
     switch read(fd, ptr, numericCast(endPtr - ptr)) {
-    case -1: throw IOError.readFailed(errno: _ss_errno())
+    case -1: throw IOError.readFailed(errno: _errno)
     case 0: throw IOError.readReachedEndOfInput
     case let n: ptr += Int(n)
     }

Sources/SwiftSyntax/SyntaxArena.swift

@@ -24,7 +24,7 @@ fileprivate struct AtomicBool {
   }
 }
 #else
-import _AtomicBool
+import _SwiftSyntaxCShims
 #endif
 
 /// A syntax arena owns the memory for all syntax nodes within it.

Sources/_AtomicBool/include/SwiftSyntaxAtomicBool.h

@@ -2,26 +2,33 @@
 //
 // This source file is part of the Swift.org open source project
 //
-// Copyright (c) 2014 - 2023 Apple Inc. and the Swift project authors
+// Copyright (c) 2014 - 2024 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
 //
 //===----------------------------------------------------------------------===//
 
-#include "SwiftSyntaxAtomicBool.h"
+#include <stdbool.h>
 
+typedef struct {
+  _Atomic(bool) value;
+} AtomicBool;
+
+__attribute__((swift_name("AtomicBool.init(initialValue:)")))
 AtomicBool atomic_bool_create(bool initialValue) {
   AtomicBool atomic;
   atomic.value = initialValue;
   return atomic;
 }
 
+__attribute__((swift_name("getter:AtomicBool.value(self:)")))
 bool atomic_bool_get(AtomicBool *atomic) {
   return atomic->value;
 }
 
+__attribute__((swift_name("setter:AtomicBool.value(self:_:)")))
 void atomic_bool_set(AtomicBool *atomic, bool newValue) {
   atomic->value = newValue;
 }

Sources/_AtomicBool/src/SwiftSyntaxAtomicBool.c renamed to Sources/_SwiftSyntaxCShims/include/_AtomicBool.h

@@ -0,0 +1,18 @@
+//===----------------------------------------------------------------------===//
+//
+// This source file is part of the Swift.org open source project
+//
+// Copyright (c) 2024 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
+//
+//===----------------------------------------------------------------------===//
+
+#include <errno.h>
+
+__attribute__((swift_name("getter:_errno()")))
+static int swiftsyntax_errno(void) {
+  return errno;
+}

Sources/_SwiftSyntaxCShims/include/_errno.h

@@ -10,23 +10,20 @@
 //
 //===----------------------------------------------------------------------===//
 
-#include <errno.h>
 #include <stdio.h>
 
-typedef FILE *_ss_ptr_FILE;
-
-static _ss_ptr_FILE _ss_stdout(void) {
+__attribute__((swift_name("getter:_stdout()")))
+static FILE *swiftsyntax_stdout(void) {
   return stdout;
 }
 
-static _ss_ptr_FILE _ss_stdin(void) {
+__attribute__((swift_name("getter:_stdin()")))
+static FILE *swiftsyntax_stdin(void) {
   return stdin;
 }
 
-static _ss_ptr_FILE _ss_stderr(void) {
+__attribute__((swift_name("getter:_stderr()")))
+static FILE *swiftsyntax_stderr(void) {
   return stderr;
 }
 
-static int _ss_errno(void) {
-  return errno;
-}

Sources/_SwiftSyntaxCShims/include/_stdio.h

@@ -1,4 +1,6 @@
 module _SwiftSyntaxCShims {
+  header "_AtomicBool.h"
+  header "_errno.h"
   header "_stdio.h"
   export *
 }