Bug 194125 - [WebGPU] Fix GPURenderPassEncoder::setVertexBuffers and allow overlapping indices with GPUBindGroups
Summary: [WebGPU] Fix GPURenderPassEncoder::setVertexBuffers and allow overlapping ind...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGPU (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Justin Fan
URL:
Keywords: InRadar
: 194182 (view as bug list)
Depends on:
Blocks:
 
Reported: 2019-01-31 16:47 PST by Justin Fan
Modified: 2019-02-04 14:55 PST (History)
10 users (show)

See Also:


Attachments
Patch (28.77 KB, patch)
2019-01-31 17:18 PST, Justin Fan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Justin Fan 2019-01-31 16:47:04 PST
[WebGPU] Fix GPURenderPassEncoder::setVertexBuffers and allow overlapping indices with GPUBindGroups
Comment 1 Justin Fan 2019-01-31 16:47:51 PST
<rdar://problem/46289645>
Comment 2 Justin Fan 2019-01-31 17:18:24 PST
Created attachment 360811 [details]
Patch
Comment 3 Myles C. Maxfield 2019-02-01 16:31:44 PST
Comment on attachment 360811 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=360811&action=review

> Source/WebCore/Modules/webgpu/WHLSL/Metal/WHLSLVertexBufferIndexCalculator.h:42
> +} // namespace Metal
> +
> +} // namespace WHLSL
> +
> +} // namespace WebCore

Ending comments unnecessary when the file is so short

> Source/WebCore/platform/graphics/gpu/cocoa/GPURenderPassEncoderMetal.mm:108
> +    [m_platformRenderPassEncoder setVertexBuffers:mtlBuffers.data() offsets:(const NSUInteger *)offsets.data() withRange:indexRanges];

wow

> Source/WebCore/platform/graphics/gpu/cocoa/GPURenderPipelineMetal.mm:33
> +#import "WHLSLVertexBufferIndexCalculator.h"

👍
Comment 4 WebKit Commit Bot 2019-02-02 11:54:01 PST
Comment on attachment 360811 [details]
Patch

Clearing flags on attachment: 360811

Committed r240898: <https://trac.webkit.org/changeset/240898>
Comment 5 WebKit Commit Bot 2019-02-02 11:54:03 PST
All reviewed patches have been landed.  Closing bug.
Comment 6 Ryan Haddad 2019-02-04 13:08:45 PST
*** Bug 194182 has been marked as a duplicate of this bug. ***
Comment 7 Justin Fan 2019-02-04 14:55:36 PST
<rdar://problem/47393543>