New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WGSL] Implement compound assignment parsing and code gen #13001
[WGSL] Implement compound assignment parsing and code gen #13001
Conversation
EWS run on previous version of this PR (hash 4b6c413) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks good, but please fix the title of the PR and fill in the bug and radar before landing.
4b6c413
to
f159612
Compare
EWS run on previous version of this PR (hash f159612) |
f159612
to
c06eb41
Compare
EWS run on current version of this PR (hash c06eb41) |
Not sure what happened to those in the original upload. |
https://bugs.webkit.org/show_bug.cgi?id=255754 rdar://problem/108347694 Reviewed by Tadeu Zagallo. Implement parsing and code gen for compound assignment statements (eg. "x += 1"). Doesn't implement type checking, which will be implemented in a follow up patch. * Source/WebGPU/WGSL/AST/ASTStringDumper.cpp: (WGSL::AST::StringDumper::visit): * Source/WebGPU/WGSL/AST/ASTStringDumper.h: * Source/WebGPU/WGSL/Lexer.cpp: (WGSL::Lexer<T>::lex): * Source/WebGPU/WGSL/Parser.cpp: (WGSL::canContinueCompoundAssignmentStatement): (WGSL::toBinaryOperation): (WGSL::Parser<Lexer>::parseStatement): * Source/WebGPU/WGSL/Token.cpp: (WGSL::toString): * Source/WebGPU/WGSL/Token.h: * Source/WebGPU/WGSL/TypeCheck.cpp: (WGSL::TypeChecker::visit): * Source/WebGPU/WGSL/tests/valid/overload.wgsl: * Tools/TestWebKitAPI/Tests/WGSL/LexerTests.cpp: (TestWGSLAPI::TEST): Canonical link: https://commits.webkit.org/263272@main
c06eb41
to
56d070d
Compare
Committed 263272@main (56d070d): https://commits.webkit.org/263272@main Reviewed commits have been landed. Closing PR #13001 and removing active labels. |
56d070d
c06eb41
π§ͺ ios-wk2π§ͺ api-macπ§ͺ ios-wk2-wptπ§ͺ gtk-wk2π§ͺ api-iosπ tvπ§ͺ mac-AS-debug-wk2π tv-simπ watchπ watch-sim