---

yaml
---
r: 346871
b: refs/heads/master
c: a9be23e
h: refs/heads/master
i:
  346869: 077500b
  346867: e039c06
  346863: 944dd1d

Author: swift-ci

[refs]

@@ -1,5 +1,5 @@
 ---
-refs/heads/master: 1fa9f9f2dcbf5b8b8d966daa3d0963a6686a5402
+refs/heads/master: a9be23e44b65d20957f94e9d117becdfa7dfe88d
 refs/heads/master-next: 203b3026584ecad859eb328b2e12490099409cd5
 refs/tags/osx-passed: b6b74147ef8a386f532cf9357a1bde006e552c54
 refs/tags/swift-2.2-SNAPSHOT-2015-12-01-a: 6bb18e013c2284f2b45f5f84f2df2887dc0f7dea

trunk/benchmark/single-source/ArrayLiteral.swift

@@ -16,7 +16,7 @@
 import TestsUtils
 
 public let ArrayLiteral = [
-  BenchmarkInfo(name: "ArrayLiteral", runFunction: run_ArrayLiteral, tags: [.validation, .api, .Array]),
+  BenchmarkInfo(name: "ArrayLiteral2", runFunction: run_ArrayLiteral, tags: [.validation, .api, .Array]),
   BenchmarkInfo(name: "ArrayValueProp", runFunction: run_ArrayValueProp, tags: [.validation, .api, .Array]),
   BenchmarkInfo(name: "ArrayValueProp2", runFunction: run_ArrayValueProp2, tags: [.validation, .api, .Array]),
   BenchmarkInfo(name: "ArrayValueProp3", runFunction: run_ArrayValueProp3, tags: [.validation, .api, .Array]),
@@ -31,7 +31,7 @@ func makeArray() -> [Int] {
 @inline(never)
 public func run_ArrayLiteral(_ N: Int) {
   for _ in 1...10000*N {
-    _ = makeArray()
+    blackHole(makeArray())
   }
 }
 

trunk/benchmark/single-source/ArraySetElement.swift

@@ -18,8 +18,7 @@ import TestsUtils
 public var ArraySetElement = BenchmarkInfo(
   name: "ArraySetElement",
   runFunction: run_ArraySetElement,
-  tags: [.runtime, .cpubench, .unstable],
-  legacyFactor: 10
+  tags: [.runtime, .cpubench]
 )
 
 // This is an effort to defeat isUniquelyReferenced optimization. Ideally
@@ -31,7 +30,7 @@ func storeArrayElement(_ array: inout [Int], _ i: Int) {
 
 public func run_ArraySetElement(_ N: Int) {
   var array = [Int](repeating: 0, count: 10000)
-  for _ in 0..<N {
+  for _ in 0..<10*N {
     for i in 0..<array.count {
       storeArrayElement(&array, i)
     }

trunk/benchmark/single-source/DataBenchmarks.swift

@@ -90,7 +90,7 @@ public let DataBenchmarks = [
     legacyFactor: 20),
 
   BenchmarkInfo(name: "DataAppendArray",
-    runFunction: { append($0*100, arraySize: 809, to: medium) }, tags: d,
+    runFunction: { append($0*100, array: array809, to: medium) }, tags: d,
     legacyFactor: 100),
 
   BenchmarkInfo(name: "DataReset",
@@ -203,6 +203,8 @@ let small = sampleData(.small)
 let medium = sampleData(.medium)
 let large = sampleData(.large)
 
+let array809 = byteArray(size: 809)
+
 let repeatElementSeq = { count in
   return sequence(state: count) { (i: inout Int) -> UInt8? in
     defer { i = i &- 1 }; return i > 0 ? UInt8(0xA0) : nil
@@ -217,10 +219,14 @@ enum SampleKind {
   case immutableBacking
 }
 
-func fillBuffer(_ buffer: UnsafeMutableBufferPointer<UInt8>) {
-  for i in buffer.indices {
-    buffer[i] = UInt8(truncatingIfNeeded: i)
+func byteArray(size: Int) -> [UInt8] {
+  var bytes = [UInt8](repeating: 0, count: size)
+  bytes.withUnsafeMutableBufferPointer { buffer in
+    for i in buffer.indices {
+      buffer[i] = UInt8(truncatingIfNeeded: i)
+    }
   }
+  return bytes
 }
 
 func sampleData(size: Int) -> Data {
@@ -235,11 +241,7 @@ func sampleData(size: Int) -> Data {
 
 func sampleBridgedNSData() -> Data {
   let count = 1033
-  var bytes = [UInt8](repeating: 0, count: count)
-  bytes.withUnsafeMutableBufferPointer {
-    fillBuffer($0)
-  }
-  let data = NSData(bytes: bytes, length: count)
+  let data = NSData(bytes: byteArray(size: count), length: count)
   return Data(referencing: data)
 }
 
@@ -295,11 +297,7 @@ func append(_ N: Int, bytes count: Int, to data: Data) {
 }
 
 @inline(never)
-func append(_ N: Int, arraySize: Int, to data: Data) {
-  var bytes = [UInt8](repeating: 0, count: arraySize)
-  bytes.withUnsafeMutableBufferPointer {
-    fillBuffer($0)
-  }
+func append(_ N: Int, array bytes: [UInt8], to data: Data) {
   for _ in 1...N {
     var copy = data
     copy.append(contentsOf: bytes)