Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 914006fabddf5fa8f9db1030c2e0a540b0c425bc
      
https://github.com/WebKit/WebKit/commit/914006fabddf5fa8f9db1030c2e0a540b0c425bc
  Author: Tadeu Zagallo <[email protected]>
  Date:   2026-02-11 (Wed, 11 Feb 2026)

  Changed paths:
    M 
LayoutTests/http/tests/webgpu/webgpu/shader/validation/expression/binary/short_circuiting_and_or-expected.txt
    M 
LayoutTests/http/tests/webgpu/webgpu/shader/validation/expression/call/builtin/clamp-expected.txt
    M 
LayoutTests/http/tests/webgpu/webgpu/shader/validation/expression/early_evaluation-expected.txt
    M Source/WebGPU/WGSL/AST/ASTCallExpression.h
    M Source/WebGPU/WGSL/AST/ASTExpression.h
    M Source/WebGPU/WGSL/ConstantFunctions.h
    M Source/WebGPU/WGSL/ConstantValue.h
    M Source/WebGPU/WGSL/EntryPointRewriter.cpp
    M Source/WebGPU/WGSL/GlobalVariableRewriter.cpp
    M Source/WebGPU/WGSL/Overload.h
    M Source/WebGPU/WGSL/TypeCheck.cpp
    M Source/WebGPU/WGSL/TypeDeclarations.rb
    M Source/WebGPU/WGSL/generator/main.rb

  Log Message:
  -----------
  [WGSL] webgpu:shader,validation,expression,call,builtin,clamp,* is failing
https://bugs.webkit.org/show_bug.cgi?id=307429
rdar://170052903

Reviewed by Mike Wyrzykowski.

We were missing the validation that `clamp(x, low, high)` should error
if `low > high`. It also required storing extra information to handle
the case where `low` and/or `high` might be override values.

We're still failing a few tests, but they appear to be invalid tests
and we're waiting for the test suite to correct them.

* 
LayoutTests/http/tests/webgpu/webgpu/shader/validation/expression/call/builtin/clamp-expected.txt:
* Source/WebGPU/WGSL/AST/ASTCallExpression.h:
* Source/WebGPU/WGSL/AST/ASTExpression.h:
(WGSL::AST::Expression::maybeEvaluation const):
(WGSL::AST::Expression::evaluation const):
* Source/WebGPU/WGSL/ConstantFunctions.h:
(WGSL::TERNARY_OPERATION):
(WGSL::VALIDATION_FUNCTION):
* Source/WebGPU/WGSL/ConstantValue.h:
(WTF::StringTypeAdapter<WGSL::ConstantValue>::StringTypeAdapter):
(WTF::StringTypeAdapter<WGSL::ConstantValue>::length const):
(WTF::StringTypeAdapter<WGSL::ConstantValue>::is8Bit const):
(WTF::StringTypeAdapter<WGSL::ConstantValue>::writeTo const):
* Source/WebGPU/WGSL/EntryPointRewriter.cpp:
(WGSL::EntryPointRewriter::checkReturnType):
* Source/WebGPU/WGSL/GlobalVariableRewriter.cpp:
(WGSL::RewriteGlobalVariables::pack):
* Source/WebGPU/WGSL/Overload.h:
* Source/WebGPU/WGSL/TypeCheck.cpp:
(WGSL::TypeChecker::visit):
(WGSL::TypeChecker::bitcast):
(WGSL::TypeChecker::evaluated):
* Source/WebGPU/WGSL/TypeDeclarations.rb:
* Source/WebGPU/WGSL/generator/main.rb:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to