[webkit-changes] [WebKit/WebKit] a8ac4a: REGRESSION (269779 at main): [ macOS iOS 17 ] 3 API t...
Tadeu Zagallo
noreply at github.com
Fri Oct 27 10:20:52 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: a8ac4a4cdd3f698758df2e2d4a053fa0676ac9f5
https://github.com/WebKit/WebKit/commit/a8ac4a4cdd3f698758df2e2d4a053fa0676ac9f5
Author: Tadeu Zagallo <tzagallo at apple.com>
Date: 2023-10-27 (Fri, 27 Oct 2023)
Changed paths:
M Source/WebGPU/WGSL/Lexer.cpp
M Tools/TestWebKitAPI/Tests/WGSL/LexerTests.cpp
Log Message:
-----------
REGRESSION (269779 at main): [ macOS iOS 17 ] 3 API tests in TestWGSL is a consistent failure/crash
https://bugs.webkit.org/show_bug.cgi?id=263746
rdar://117553256
Reviewed by Mike Wyrzykowski.
We had 4 test cases failing, which consisted of 3 incorrect tests and 1 edge case:
- The edge case was when we parse a floating point at the end of the input source,
ending with a period (the test was `01.`). We have to run the state machine once
more past the end of the file to read the EOF char and either return an error or
terminate a valid number token.
- 2 tests were parsing numbers that had the `f` suffix in the WGSL source, but
comparing against a double value in C++. Originally the lexer did not perform
the narrowing to float, so the comparison passed, but it shouldn't, so I updated
the expectations.
- The last failing test was an invalid input (`42e-4`), which now correctly
returns an invalid token.
* Source/WebGPU/WGSL/Lexer.cpp:
(WGSL::Lexer<T>::lexNumber):
* Tools/TestWebKitAPI/Tests/WGSL/LexerTests.cpp:
(TestWGSLAPI::TEST):
Canonical link: https://commits.webkit.org/269856@main
More information about the webkit-changes
mailing list