[stdlib] remove deinitialize branch (does this do anything?)

Author: oxy

stdlib/public/core/ArrayBufferProtocol.swift

@@ -160,9 +160,9 @@ extension _ArrayBufferProtocol {
     let holeStart = elements + subrange.lowerBound
     let holeEnd = holeStart + newCount
 
-    if eraseCount > 0 {
-      holeStart.deinitialize(count: eraseCount)
-    }
+    // directly forwards to Builtin.destroyArray
+    // TODO: should we branch here or does the compiler branch implicitly?
+    holeStart.deinitialize(count: eraseCount)
 
     // resize the hole to make it the correct size
     // safe to always call, moveInitialize to the same location is a no-op