[webkit-changes] [WebKit/WebKit] f425f4: [WGSL] Concretizing atomic types shouldn't assert

Tadeu Zagallo noreply at github.com
Fri Oct 20 13:00:53 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f425f41fe590130e789e92b8ad71e0d54d69a2a6
      https://github.com/WebKit/WebKit/commit/f425f41fe590130e789e92b8ad71e0d54d69a2a6
  Author: Tadeu Zagallo <tzagallo at apple.com>
  Date:   2023-10-20 (Fri, 20 Oct 2023)

  Changed paths:
    M Source/WebGPU/WGSL/Constraints.cpp
    M Source/WebGPU/WGSL/tests/valid/concretization.wgsl

  Log Message:
  -----------
  [WGSL] Concretizing atomic types shouldn't assert
https://bugs.webkit.org/show_bug.cgi?id=263402
rdar://117223889

Reviewed by Dan Glastonbury.

When concretization was originally implemented it was only used when initializing
variables, and since variables can't initialized with atomic values, it just asserted
that we never tried to concretize an atomic type. However, we also need to concretize
array elements when accessing an array with a non-constant index. That was implemented
in 269416 at main, and now we can crash if indexing into array of atomics. The fix is simple,
concretizing an atomic type should just return the same type.

* Source/WebGPU/WGSL/Constraints.cpp:
(WGSL::concretize):
* Source/WebGPU/WGSL/tests/valid/concretization.wgsl:

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




More information about the webkit-changes mailing list