---

yaml
---
r: 260095
b: refs/heads/swift-5.0-branch
c: 83ed42a
h: refs/heads/master
i:
  260093: 86487e3
  260091: 6ecd7d4
  260087: 973ad33
  260079: 8e42594
  260063: e06d57f
  260031: 1bde991
  259967: 223dfc1
  259839: ebe98c4
  259583: ac1ea05
  259071: 10b96b9
  258047: 9e0348f

Author: milseman

[refs]

@@ -642,7 +642,7 @@ refs/tags/swift-DEVELOPMENT-SNAPSHOT-2017-12-22-a: 5a2d73c274f7917cce75d57bdd7b7
 refs/tags/swift-DEVELOPMENT-SNAPSHOT-2017-12-23-a: b7c074342459a645779f106c42bf4852345ed413
 refs/heads/master-llvm-swift5-transition: 8ace18c8953afb3d7d94cf04cacc0b51a7e5f1e3
 refs/heads/revert-13597-master: cccee1df039d072215f9bddc2cbc1e32a8d5d5ee
-refs/heads/swift-5.0-branch: 953d5e8dcf9f26a25374067f46fe80e6178b6ebd
+refs/heads/swift-5.0-branch: 83ed42a6023e7840a5a72766cbbfb2aa50754ae5
 refs/tags/swift-4.1-DEVELOPMENT-SNAPSHOT-2017-12-23-a: b32214f7e04339dfada623b6b76dbebfb41e4541
 refs/tags/swift-4.1-DEVELOPMENT-SNAPSHOT-2017-12-24-a: 1eb0be506c0744c7eff0550a10240286046e181d
 refs/tags/swift-4.1-DEVELOPMENT-SNAPSHOT-2017-12-25-a: f35a91502bad0065c83d8760407c23be7b899f48

branches/swift-5.0-branch/stdlib/public/core/StringGutsSlice.swift

@@ -16,22 +16,18 @@
 
 // A sliced _StringGuts, convenient for unifying String/Substring comparison,
 // hashing, and RRC.
-@_fixed_layout
-@usableFromInline
 internal struct _StringGutsSlice {
-  @usableFromInline
   internal var _guts: _StringGuts
 
-  @usableFromInline
   internal var _offsetRange: Range<Int>
 
-  @inlinable @inline(__always)
+  @inline(__always)
   internal init(_ guts: _StringGuts) {
     self._guts = guts
     self._offsetRange = 0..<self._guts.count
   }
 
-  @inlinable @inline(__always)
+  @inline(__always)
   internal init(_ guts: _StringGuts, _ offsetRange: Range<Int>) {
     self._guts = guts
     self._offsetRange = offsetRange
@@ -83,7 +79,7 @@ internal struct _StringGutsSlice {
     }
   }
 
-  @inlinable @inline(__always)
+  @inline(__always)
   internal func withFastUTF8<R>(
     _ f: (UnsafeBufferPointer<UInt8>) throws -> R
   ) rethrows -> R {

branches/swift-5.0-branch/stdlib/public/core/StringHashable.swift

@@ -45,7 +45,6 @@ extension StringProtocol {
 }
 
 extension _StringGutsSlice {
-  @usableFromInline // @opaque
   @inline(never) // slow-path
   internal func _normalizedHash(into hasher: inout Hasher) {
     if self.isNFCFastUTF8 {

branches/swift-5.0-branch/stdlib/public/core/StringProtocol.swift

@@ -132,7 +132,6 @@ extension StringProtocol {
     get { return String(self) }
   }
 
-  @inlinable // Eliminate for String, Substring
   internal var _gutsSlice: _StringGutsSlice {
     @_specialize(where Self == String)
     @_specialize(where Self == Substring)
@@ -152,7 +151,8 @@ extension StringProtocol {
     @inline(__always) get {
       let start = startIndex
       let end = endIndex
-      _internalInvariant(start.transcodedOffset == 0 && end.transcodedOffset == 0)
+      _internalInvariant(
+        start.transcodedOffset == 0 && end.transcodedOffset == 0)
       return Range(uncheckedBounds: (start.encodedOffset, end.encodedOffset))
     }
   }

branches/swift-5.0-branch/stdlib/public/core/StringStorage.swift

@@ -189,7 +189,6 @@ final internal class _StringStorage: _AbstractStringStorage {
 
   internal var _reserved: UInt16
 
-  @inlinable
   override internal var count: Int {
     @inline(__always) get { return _count }
     @inline(__always) set { _count = newValue }

branches/swift-5.0-branch/stdlib/public/core/StringUTF16View.swift

@@ -537,6 +537,8 @@ extension String {
   ) {
     _internalInvariant(_guts.isFastUTF8)
 
+    if _slowPath(range.isEmpty) { return }
+
     return _guts.withFastUTF8 { utf8 in
       var writeIdx = 0
       let writeEnd = buffer.count

branches/swift-5.0-branch/test/api-digester/Outputs/stability-stdlib-abi.swift.expected

@@ -483,6 +483,23 @@ Protocol _NSEnumerator has been removed
 Protocol _NSFastEnumeration has been removed
 Protocol _ShadowProtocol has been removed
 
+Constructor _StringGutsSlice.init(_:) has been removed
+Constructor _StringGutsSlice.init(_:_:) has been removed
+Func _StringGutsSlice._normalizedHash(into:) has been removed
+Func _StringGutsSlice.compare(with:expecting:) has been removed
+Func _StringGutsSlice.withFastUTF8(_:) has been removed
+Struct _StringGutsSlice is now without @_fixed_layout
+Var StringProtocol._gutsSlice has been removed
+Var _StringGutsSlice._guts has been removed
+Var _StringGutsSlice._offsetRange has been removed
+Var _StringGutsSlice.count has been removed
+Var _StringGutsSlice.end has been removed
+Var _StringGutsSlice.isASCII has been removed
+Var _StringGutsSlice.isFastUTF8 has been removed
+Var _StringGutsSlice.isNFCFastUTF8 has been removed
+Var _StringGutsSlice.range has been removed
+Var _StringGutsSlice.start has been removed
+
 Func ManagedBufferPointer._sanityCheckValidBufferClass(_:creating:) has been removed
 Func _sanityCheck(_:_:file:line:) has been removed
 Func _sanityCheckFailure(_:file:line:) has been removed

branches/swift-5.0-branch/validation-test/stdlib/StringBreadcrumbs.swift

@@ -38,14 +38,17 @@ let StringBreadcrumbsTests = TestSuite("StringBreadcrumbsTests")
 
 func validateBreadcrumbs(_ str: String) {
   var utf16CodeUnits = Array(str.utf16)
-  var utf16Indices = Array(str.utf16.indices)
   var outputBuffer = Array<UInt16>(repeating: 0, count: utf16CodeUnits.count)
 
-  for i in 0..<(utf16CodeUnits.count-1) {
-    for j in (i+1)..<utf16CodeUnits.count {
+  // Include the endIndex, so we can test end conversions
+  var utf16Indices = Array(str.utf16.indices) + [str.utf16.endIndex]
+
+  for i in 0...utf16CodeUnits.count {
+    for j in i...utf16CodeUnits.count {
       let range = Range(uncheckedBounds: (i, j))
 
       let indexRange = str._toUTF16Indices(range)
+
       // Range<String.Index> <=> Range<Int>
       expectEqual(utf16Indices[i], indexRange.lowerBound)
       expectEqual(utf16Indices[j], indexRange.upperBound)
@@ -69,6 +72,13 @@ func validateBreadcrumbs(_ str: String) {
   }
 }
 
+StringBreadcrumbsTests.test("uniform strings") {
+  validateBreadcrumbs(smallASCII)
+  validateBreadcrumbs(largeASCII)
+  validateBreadcrumbs(smallUnicode)
+  validateBreadcrumbs(largeUnicode)
+}
+
 StringBreadcrumbsTests.test("largeString") {
   validateBreadcrumbs(largeString)
 }