[webkit-changes] [WebKit/WebKit] ca3820: [WebGPU] bind group dynamic offsets are not implem...

mwyrzykowski noreply at github.com
Thu Dec 28 13:31:26 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ca382024b6d3ecf1dc7042ed2d260d58fd1aa5c1
      https://github.com/WebKit/WebKit/commit/ca382024b6d3ecf1dc7042ed2d260d58fd1aa5c1
  Author: Mike Wyrzykowski <mwyrzykowski at apple.com>
  Date:   2023-12-28 (Thu, 28 Dec 2023)

  Changed paths:
    M Source/WebGPU/WebGPU/BindGroupLayout.h
    M Source/WebGPU/WebGPU/BindGroupLayout.mm
    M Source/WebGPU/WebGPU/PipelineLayout.mm

  Log Message:
  -----------
  [WebGPU] bind group dynamic offsets are not implemented correctly, they don't correspond to the proper stage
https://bugs.webkit.org/show_bug.cgi?id=266815>
<radar://120049732>

Reviewed by Tadeu Zagallo.

Dynamic offsets assumed buffers had linearly increasing bind
group indices, which does not necessarily hold and is not
the case in Unity's demos.

Break this assumption by computing the corresponding index
from the source dynamic offsets buffer during bind group
layout generation time.

Then use this precomputed index instead of a linearly increasing
value in the PipelineLayout.

* Source/WebGPU/WebGPU/BindGroupLayout.h:
* Source/WebGPU/WebGPU/BindGroupLayout.mm:
(WebGPU::Device::createBindGroupLayout):
* Source/WebGPU/WebGPU/PipelineLayout.mm:
(WebGPU::PipelineLayout::offsetVectorForBindGroup):

Canonical link: https://commits.webkit.org/272522@main




More information about the webkit-changes mailing list