[webkit-changes] [WebKit/WebKit] b1db59: [WGSL] Add more logging to constant rewriter and c...

Tadeu Zagallo noreply at github.com
Fri Sep 29 01:08:43 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b1db598cbd01a8635da27a489372bc5f945dd4e5
      https://github.com/WebKit/WebKit/commit/b1db598cbd01a8635da27a489372bc5f945dd4e5
  Author: Tadeu Zagallo <tzagallo at apple.com>
  Date:   2023-09-29 (Fri, 29 Sep 2023)

  Changed paths:
    M Source/WebGPU/WGSL/ConstantRewriter.cpp

  Log Message:
  -----------
  [WGSL] Add more logging to constant rewriter and clear evaluation result
https://bugs.webkit.org/show_bug.cgi?id=262305
rdar://116179924

Reviewed by Mike Wyrzykowski.

There was a bug in the constant rewriter where the wrong value would be returned
for expressions. This is caused by the fact that the rewriter implementation is
rather sparse at the moment, and we weren't cleaning the previous evaluation result,
so a stale value was being returned. In order to identify the root cause of the
issue more logging was required, and it felt appropriate to leave the logging in
(behind the same flag already in use) for future debugging.

* Source/WebGPU/WGSL/ConstantRewriter.cpp:
(WGSL::ConstantRewriter::visit):
(WGSL::ConstantRewriter::evaluate):
(WGSL::ConstantRewriter::evaluated):
(WGSL::ConstantRewriter::introduceVariable):

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




More information about the webkit-changes mailing list