WHLSL Metal code generation unnecessarily does string copies by passing partial results as Strings
Created attachment 375632 [details] Patch
Comment on attachment 375632 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=375632&action=review > Source/WebCore/platform/graphics/gpu/cocoa/GPUComputePipelineMetal.mm:93 > + // FIXME: Add direct StringBuilder -> NSString conversion to void extra copy into a WTF::String void => avoid Maybe also worth filing a bug? > Source/WebCore/platform/graphics/gpu/cocoa/GPURenderPipelineMetal.mm:405 > + // FIXME: Add direct StringBuilder -> NSString conversion to void extra copy into a WTF::String ditto
(In reply to Saam Barati from comment #2) > Comment on attachment 375632 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=375632&action=review > > > Source/WebCore/platform/graphics/gpu/cocoa/GPUComputePipelineMetal.mm:93 > > + // FIXME: Add direct StringBuilder -> NSString conversion to void extra copy into a WTF::String > > void => avoid > > Maybe also worth filing a bug? https://bugs.webkit.org/show_bug.cgi?id=200474
Committed r248307: <https://trac.webkit.org/changeset/248307>
<rdar://problem/53997291>