Remove some unnecessary changes from the diff

Author: grynspan

Sources/Testing/ExitTests/ExitTest.swift

@@ -438,7 +438,7 @@ extension ExitTest {
     // External tools authors should set up their own back channel mechanisms
     // and ensure they're installed before calling ExitTest.callAsFunction().
     guard var result = find(at: sourceLocation) else {
-      fatalError("Could not find an exit test that should have been located at \(sourceLocation).")
+      return nil
     }
 
     // We can't say guard let here because it counts as a consume.

Sources/Testing/SourceAttribution/Expression.swift

@@ -22,8 +22,9 @@
 /// let swiftSyntaxExpr: ExprSyntax = "\(testExpr)"
 /// ```
 ///
-/// - Warning: This type is used to implement the `#expect()` macro. Do not use
-///   it directly. Tools can use the SPI ``Expression`` typealias if needed.
+/// - Warning: This type is used to implement the `#expect(exitsWith:)`
+///   macro. Do not use it directly. Tools can use the SPI ``Expression``
+///   typealias if needed.
 public struct __Expression: Sendable {
   /// An enumeration describing the various kinds of expression that can be
   /// captured.

Sources/Testing/Support/Additions/NumericAdditions.swift

@@ -34,19 +34,3 @@ extension UInt8 {
     self == UInt8(ascii: "\r") || self == UInt8(ascii: "\n")
   }
 }
-
-// MARK: -
-
-extension Int {
-  /// Get the next integer after this one that is properly aligned to store a
-  /// value of type `T`.
-  ///
-  /// - Parameters:
-  ///   - type: the type whose alignment should be used to compute the result.
-  ///
-  /// - Returns: an integer greater than or equal to `self` whose value is
-  ///   properly aligned to store a value of type `T`.
-  func alignedUp(for type: (some Any).Type) -> Self {
-    Self(bitPattern: UnsafeRawPointer(bitPattern: self)?.alignedUp(for: type))
-  }
-}

Sources/Testing/Test+Macro.swift

@@ -8,10 +8,6 @@
 // See https://swift.org/CONTRIBUTORS.txt for Swift project authors
 //
 
-/// This file provides support for the `@Test` macro. Other than the macro
-/// itself, the symbols in this file should not be used directly and are subject
-/// to change as the testing library evolves.
-
 #if _runtime(_ObjC)
 public import ObjectiveC
 
@@ -46,6 +42,10 @@ public typealias __XCTestCompatibleSelector = Never
 #endif
 }
 
+/// This file provides support for the `@Test` macro. Other than the macro
+/// itself, the symbols in this file should not be used directly and are subject
+/// to change as the testing library evolves.
+
 // MARK: - @Suite
 
 /// Declare a test suite.
@@ -499,8 +499,8 @@ extension Test {
 /// - Note: This macro has compile-time effects _only_ and should not affect a
 ///   compiled test target.
 ///
-/// - Warning: This macro is used to implement other macros declared by the
-///   testing library. Do not use it directly.
+/// - Warning: This macro is used to implement other macros declared by the testing
+///   library. Do not use it directly.
 @attached(peer) public macro __testing(
   semantics arguments: _const String...
 ) = #externalMacro(module: "TestingMacros", type: "PragmaMacro")

Sources/TestingMacros/ConditionMacro.swift

@@ -478,7 +478,6 @@ extension ExitTestConditionMacro {
 
 #if !SWT_NO_LEGACY_TEST_DISCOVERY
     // Emit a legacy type declaration if SymbolLinkageMarkers is off.
-    // TODO: remove this decl when we drop support for emitting legacy metadata
     let legacyEnumName = context.makeUniqueName("__🟠$exit_test_body__")
     decls.append(
       """

Sources/TestingMacros/SuiteDeclarationMacro.swift

@@ -169,7 +169,6 @@ public struct SuiteDeclarationMacro: MemberMacro, PeerMacro, Sendable {
     // also annotated unavailable, since it's meant only for use by the testing
     // library at runtime. The compiler does not allow combining 'unavailable'
     // and 'deprecated' into a single availability attribute: rdar://111329796
-    // TODO: remove this decl when we drop support for emitting legacy metadata
     let typeName = declaration.type.tokens(viewMode: .fixedUp).map(\.textWithoutBackticks).joined()
     let enumName = context.makeUniqueName("__🟠$test_container__suite__\(typeName)")
     result.append(

Sources/TestingMacros/TestDeclarationMacro.swift

@@ -484,7 +484,6 @@ public struct TestDeclarationMacro: PeerMacro, Sendable {
     // by the testing library at runtime. The compiler does not allow combining
     // 'unavailable' and 'deprecated' into a single availability attribute:
     // rdar://111329796
-    // TODO: remove this decl when we drop support for emitting legacy metadata
     let enumName = context.makeUniqueName(thunking: functionDecl, withPrefix: "__🟠$test_container__function__")
     result.append(
       """

Sources/_TestingInternals/include/Includes.h

@@ -96,10 +96,6 @@
 #include <limits.h>
 #endif
 
-#if __has_include(<link.h>)
-#include <link.h>
-#endif
-
 #if __has_include(<spawn.h>)
 #include <spawn.h>
 #endif