[stdlib] remove bulk-update from MutableSpan per SE-0485

Author: glessard

stdlib/public/core/Span/MutableSpan.swift

@@ -355,56 +355,6 @@ extension MutableSpan where Element: ~Copyable {
   }
 }
 
-@available(SwiftCompatibilitySpan 5.0, *)
-@_originallyDefinedIn(module: "Swift;CompatibilitySpan", SwiftCompatibilitySpan 6.2)
-extension MutableSpan where Element: BitwiseCopyable {
-
-  /// Accesses the element at the specified position in the `Span`.
-  ///
-  /// - Parameter position: The offset of the element to access. `position`
-  ///     must be greater or equal to zero, and less than `count`.
-  ///
-  /// - Complexity: O(1)
-  @_alwaysEmitIntoClient
-  public subscript(_ position: Index) -> Element {
-    get {
-      _precondition(indices.contains(position), "index out of bounds")
-      return unsafe self[unchecked: position]
-    }
-    @lifetime(self: copy self)
-    set {
-      _precondition(indices.contains(position), "index out of bounds")
-      unsafe self[unchecked: position] = newValue
-    }
-  }
-
-  /// Accesses the element at the specified position in the `Span`.
-  ///
-  /// This subscript does not validate `position`; this is an unsafe operation.
-  ///
-  /// - Parameter position: The offset of the element to access. `position`
-  ///     must be greater or equal to zero, and less than `count`.
-  ///
-  /// - Complexity: O(1)
-  @unsafe
-  @_alwaysEmitIntoClient
-  public subscript(unchecked position: Index) -> Element {
-    get {
-      let offset = position&*MemoryLayout<Element>.stride
-      return unsafe _start().loadUnaligned(
-        fromByteOffset: offset, as: Element.self
-      )
-    }
-    @lifetime(self: copy self)
-    set {
-      let offset = position&*MemoryLayout<Element>.stride
-      unsafe _start().storeBytes(
-        of: newValue, toByteOffset: offset, as: Element.self
-      )
-    }
-  }
-}
-
 @available(SwiftCompatibilitySpan 5.0, *)
 @_originallyDefinedIn(module: "Swift;CompatibilitySpan", SwiftCompatibilitySpan 6.2)
 extension MutableSpan where Element: ~Copyable {
@@ -476,220 +426,6 @@ extension MutableSpan {
       unsafe $0.update(repeating: repeatedValue, count: count)
     }
   }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update<S: Sequence>(
-    from source: S
-  ) -> (unwritten: S.Iterator, index: Index) where S.Element == Element {
-    var iterator = source.makeIterator()
-    let index = update(from: &iterator)
-    return (iterator, index)
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(
-    from elements: inout some IteratorProtocol<Element>
-  ) -> Index {
-    var index = 0
-    while index < _count {
-      guard let element = elements.next() else { break }
-      unsafe self[unchecked: index] = element
-      index &+= 1
-    }
-    return index
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(
-    fromContentsOf source: some Collection<Element>
-  ) -> Index {
-    let updated = source.withContiguousStorageIfAvailable {
-      self.update(fromContentsOf: unsafe Span(_unsafeElements: $0))
-    }
-    if let updated {
-      return updated
-    }
-
-    //TODO: use _copyContents here
-
-    var iterator = source.makeIterator()
-    let index = update(from: &iterator)
-    _precondition(
-      iterator.next() == nil,
-      "destination buffer view cannot contain every element from source."
-    )
-    return index
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(fromContentsOf source: Span<Element>) -> Index {
-    guard !source.isEmpty else { return 0 }
-    _precondition(
-      source.count <= self.count,
-      "destination span cannot contain every element from source."
-    )
-    unsafe _start().withMemoryRebound(
-      to: Element.self, capacity: source.count
-    ) { dest in
-      unsafe source.withUnsafeBufferPointer {
-        unsafe dest.update(from: $0.baseAddress!, count: $0.count)
-      }
-    }
-    return source.count
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(
-    fromContentsOf source: borrowing MutableSpan<Element>
-  ) -> Index {
-    update(fromContentsOf: source.span)
-  }
-}
-
-@available(SwiftCompatibilitySpan 5.0, *)
-@_originallyDefinedIn(module: "Swift;CompatibilitySpan", SwiftCompatibilitySpan 6.2)
-extension MutableSpan where Element: ~Copyable {
-
-//  @_alwaysEmitIntoClient
-//  public mutating func moveUpdate(
-//    fromContentsOf source: consuming OutputSpan<Element>
-//  ) -> Index {
-//    guard !source.isEmpty else { return 0 }
-//    _precondition(
-//      source.count <= self.count,
-//      "destination span cannot contain every element from source."
-//    )
-//    let buffer = unsafe source.relinquishBorrowedMemory()
-//    // we must now deinitialize the returned UMBP
-//    unsafe _start().moveInitializeMemory(
-//      as: Element.self, from: buffer.baseAddress!, count: buffer.count
-//    )
-//    return buffer.count
-//  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func moveUpdate(
-    fromContentsOf source: UnsafeMutableBufferPointer<Element>
-  ) -> Index {
-//    let source = OutputSpan(_initializing: source, initialized: source.count)
-//    return self.moveUpdate(fromContentsOf: source)
-    unsafe withUnsafeMutableBufferPointer {
-      unsafe $0.moveUpdate(fromContentsOf: source)
-    }
-  }
-}
-
-@available(SwiftCompatibilitySpan 5.0, *)
-@_originallyDefinedIn(module: "Swift;CompatibilitySpan", SwiftCompatibilitySpan 6.2)
-extension MutableSpan {
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func moveUpdate(
-    fromContentsOf source: Slice<UnsafeMutableBufferPointer<Element>>
-  ) -> Index {
-    unsafe moveUpdate(fromContentsOf: .init(rebasing: source))
-  }
-}
-
-@available(SwiftCompatibilitySpan 5.0, *)
-@_originallyDefinedIn(module: "Swift;CompatibilitySpan", SwiftCompatibilitySpan 6.2)
-extension MutableSpan where Element: BitwiseCopyable {
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(
-    repeating repeatedValue: Element
-  ) where Element: BitwiseCopyable {
-    guard count > 0 else { return }
-    // rebind _start manually in order to avoid assumptions about alignment.
-    let rp = unsafe _start()._rawValue
-    let binding = Builtin.bindMemory(rp, count._builtinWordValue, Element.self)
-    let rebound = unsafe UnsafeMutablePointer<Element>(rp)
-    unsafe rebound.update(repeating: repeatedValue, count: count)
-    Builtin.rebindMemory(rp, binding)
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update<S: Sequence>(
-    from source: S
-  ) -> (unwritten: S.Iterator, index: Index)
-  where S.Element == Element, Element: BitwiseCopyable {
-    var iterator = source.makeIterator()
-    let index = update(from: &iterator)
-    return (iterator, index)
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(
-    from elements: inout some IteratorProtocol<Element>
-  ) -> Index {
-    var index = 0
-    while index < _count {
-      guard let element = elements.next() else { break }
-      unsafe self[unchecked: index] = element
-      index &+= 1
-    }
-    return index
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(
-    fromContentsOf source: some Collection<Element>
-  ) -> Index where Element: BitwiseCopyable {
-    let updated = source.withContiguousStorageIfAvailable {
-      self.update(fromContentsOf: unsafe Span(_unsafeElements: $0))
-    }
-    if let updated {
-      return updated
-    }
-
-    //TODO: use _copyContents here
-
-    var iterator = source.makeIterator()
-    let index = update(from: &iterator)
-    _precondition(
-      iterator.next() == nil,
-      "destination buffer view cannot contain every element from source."
-    )
-    return index
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(
-    fromContentsOf source: Span<Element>
-  ) -> Index where Element: BitwiseCopyable {
-    guard !source.isEmpty else { return 0 }
-    _precondition(
-      source.count <= self.count,
-      "destination span cannot contain every element from source."
-    )
-    unsafe source.withUnsafeBufferPointer {
-      unsafe _start().copyMemory(
-        from: $0.baseAddress!,
-        byteCount: $0.count &* MemoryLayout<Element>.stride
-      )
-    }
-    return source.count
-  }
-
-  @_alwaysEmitIntoClient
-  @lifetime(self: copy self)
-  public mutating func update(
-    fromContentsOf source: borrowing MutableSpan<Element>
-  ) -> Index where Element: BitwiseCopyable {
-    update(fromContentsOf: source.span)
-  }
 }
 
 // MARK: sub-spans