<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Wouldn't materializeForSet be good enough? I guess that's <i class="">one</i>&nbsp;extra copy if you don't want to rely on things not guaranteed by the language (i.e. "fields of classes have stable addresses").</div><div class=""><br class=""></div><div class="">Jordan</div><div class=""><br class=""></div><br class=""><div><blockquote type="cite" class=""><div class="">On Mar 10, 2016, at 7:43 , Arnold Schwaighofer &lt;<a href="mailto:aschwaighofer@apple.com" class="">aschwaighofer@apple.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Nice!<br class=""><br class="">As you have mentioned, unfortunately, this does not work on nested types without spurious copies (e.g. for foo.left.value.right.value = …). You would need to implement this with a mutableAddressor. Unfortunately, doing so requires Builtins and so this can’t currently be done outside of the standard library (without nasty flags).<br class=""><br class=""><span id="cid:E31920EA-99D2-47EC-AC60-6E50BD8FDB7A@localhost">&lt;0001-A-copy-on-write-data-type.patch&gt;</span><br class=""><br class="">I agree with you that some interface forwarding would be an awesome language feature to have.<br class=""><br class="">It would allow for implementing an indirect storage wrapper without lots of boilerplate.<br class=""><br class="">Consider an example where you have value types that implement a protocol:<br class=""><br class=""><br class="">public _MyProtoWithoutAsscociatedType {<br class="">}<br class=""><br class="">public protocol MyProtocol : MyProtoWithoutAsscociatedType {<br class=""> &nbsp;associatedtype Assoc<br class=""><br class=""> &nbsp;public func doThis(v : Assoc)<br class=""> &nbsp;public func doThat()<br class=""><br class=""> &nbsp;public static func initAssocType() -&gt; Assoc<br class=""><br class="">}<br class=""><br class="">struct MyProtocolImplementation : MyProtocol {<br class=""> &nbsp;typealias Assoc = Int<br class=""> &nbsp;var a : AReference<br class=""> &nbsp;var b : AReference<br class=""> &nbsp;var c : AReference<br class=""> &nbsp;var d : Double<br class=""><br class=""> &nbsp;func doThis(v: Assoc) {…}<br class=""> &nbsp;func doThat() {…}<br class=""><br class=""> &nbsp;static func initAssocType() -&gt; Assoc {<br class=""> &nbsp;&nbsp;&nbsp;return 0<br class=""> &nbsp;}<br class="">}<br class=""><br class="">Your application passes those value types as existential values or as generic values because the architecture requires that. This can incur lots of allocations at every value copy for this example value type because we don’t fit in the inline buffer (three pointer words) which would be allocated inline (on the stack or as a stored property for existentials).<br class=""><br class="">One way to work around this is today is make your value type’s storage indirect via a wrapper such that it will always fit in inline storage.<br class=""><br class="">class _BoxStorage&lt;T&gt; {<br class=""> &nbsp;&nbsp;&nbsp;final var value: T<br class=""><br class=""> &nbsp;&nbsp;&nbsp;init(_ elt: T) {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;value = elt<br class=""> &nbsp;&nbsp;&nbsp;}<br class="">}<br class=""><br class="">public struct Indirect&lt;V : MyProtocol&gt; : MyProcotol {<br class=""><br class=""> &nbsp;&nbsp;&nbsp;public typealias Assoc = V.Assoc<br class=""><br class=""> &nbsp;&nbsp;&nbsp;final var _storage: _BoxStorage&lt;V&gt;<br class=""><br class=""> &nbsp;&nbsp;&nbsp;public init(_ elt: V) {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;_storage = _BoxStorage(elt)<br class=""> &nbsp;&nbsp;&nbsp;}<br class=""><br class=""> &nbsp;&nbsp;&nbsp;internal var value: V {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;get {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return _storage.value<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;mutableAddressWithNativeOwner {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if !isUniquelyReferencedNonObjC(&amp;_storage) {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;_storage = _BoxStorage(_storage.value)<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return (<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;UnsafeMutablePointer(Builtin.addressof(&amp;_storage.value)),<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Builtin.castToNativeObject(_storage))<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br class=""> &nbsp;&nbsp;&nbsp;}<br class=""><br class=""> &nbsp;&nbsp;&nbsp;public static func initAssocType() -&gt; Assoc {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return Assoc.initAssocType()<br class=""> &nbsp;&nbsp;&nbsp;}<br class=""><br class=""> &nbsp;&nbsp;&nbsp;public func doThis(v: Assoc) {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;value.doThis(v)<br class=""> &nbsp;&nbsp;&nbsp;}<br class=""> &nbsp;&nbsp;&nbsp;public func doThat() {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;value.doThat()<br class=""> &nbsp;&nbsp;&nbsp;}<br class="">}<br class=""><br class="">Indeed, doing so would require a lot of boiler plate.<br class=""><br class=""><br class="">Instead the following would be much nicer:<br class=""><br class="">public struct Indirect&lt;V : MyProtocol&gt; : MyProcotol {<br class=""><br class=""> &nbsp;&nbsp;&nbsp;public typealias Assoc = V.Assoc<br class=""><br class=""> &nbsp;&nbsp;&nbsp;final var _storage: _BoxStorage&lt;V&gt;<br class=""><br class=""> &nbsp;&nbsp;&nbsp;public init(_ elt: V) {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;_storage = _BoxStorage(elt)<br class=""> &nbsp;&nbsp;&nbsp;}<br class=""><br class=""> &nbsp;&nbsp;&nbsp;internal forwarding var value: V as MyProtocol {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;typealias Assoc = V.Assoc<br class=""><br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;get {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return _storage.value<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;mutableAddressWithNativeOwner {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if !isUniquelyReferencedNonObjC(&amp;_storage) {<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;_storage = _BoxStorage(_storage.value)<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return (<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;UnsafeMutablePointer(Builtin.addressof(&amp;_storage.value)),<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Builtin.castToNativeObject(_storage))<br class=""> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br class=""> &nbsp;&nbsp;&nbsp;}<br class=""><br class="">}<br class=""><br class=""><br class=""><br class=""><blockquote type="cite" class="">On Mar 9, 2016, at 9:53 AM, Jordan Rose via swift-dev &lt;<a href="mailto:swift-dev@swift.org" class="">swift-dev@swift.org</a>&gt; wrote:<br class=""><br class="">Ha, I remember this document, but forgot it included an implementation. Encouraging to see that it's pretty much identical.<br class=""><br class="">The concrete type forwarding logic is something we ought to be able to simplify somehow (with new language features).<br class=""><br class="">Jordan<br class=""><br class=""><br class=""><blockquote type="cite" class="">On Mar 9, 2016, at 9:50 , Nadav Rotem &lt;<a href="mailto:nrotem@apple.com" class="">nrotem@apple.com</a>&gt; wrote:<br class=""><br class="">HI Jordan, <br class=""><br class="">Very Nice! &nbsp;&nbsp;&nbsp;<br class=""><br class="">There is a similar implementation and discussion about performance here:<br class=""><br class=""><a href="https://github.com/apple/swift/blob/master/docs/OptimizationTips.rst#advice-use-copy-on-write-semantics-for-large-values" class="">https://github.com/apple/swift/blob/master/docs/OptimizationTips.rst#advice-use-copy-on-write-semantics-for-large-values</a><br class=""><br class="">-Nadav<br class=""><br class=""><br class=""><blockquote type="cite" class="">On Mar 9, 2016, at 9:39 AM, Jordan Rose via swift-dev &lt;swift-dev@swift.org&gt; wrote:<br class=""><br class="">Just for fun, I wrote a wrapper for COW types that don't need the flexible inline storage of ManagedBuffer. It turned out to be pretty straightforward, though I didn't bother with materializeForSet and thus am incurring the cost of many extra value copies on mutation. The major downside is having to forward all operations through, something I'm sure the implementers of Array and Dictionary are very used to!<br class=""><br class="">Disclaimer: This is not performant; don't use it in your app.<br class=""><br class="">Jordan<br class=""><br class="">&lt;cow.swift&gt;<br class="">_______________________________________________<br class="">swift-dev mailing list<br class="">swift-dev@swift.org<br class="">https://lists.swift.org/mailman/listinfo/swift-dev<br class=""></blockquote><br class=""></blockquote><br class="">_______________________________________________<br class="">swift-dev mailing list<br class=""><a href="mailto:swift-dev@swift.org" class="">swift-dev@swift.org</a><br class="">https://lists.swift.org/mailman/listinfo/swift-dev<br class=""></blockquote><br class=""></div></div></blockquote></div><br class=""></body></html>