-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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] Add support for struct constructors #13807
[WGSL] Add support for struct constructors #13807
Conversation
EWS run on current version of this PR (hash 02108de) |
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.
Looks right.
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.
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.
π
https://bugs.webkit.org/show_bug.cgi?id=256704 rdar://109265280 Reviewed by Myles C. Maxfield and Dan Glastonbury. Add type checking and code generation for struct constructors. * Source/WebGPU/WGSL/Metal/MetalFunctionWriter.cpp: (WGSL::Metal::FunctionDefinitionWriter::visit): * Source/WebGPU/WGSL/TypeCheck.cpp: (WGSL::TypeChecker::visit): * Source/WebGPU/WGSL/tests/invalid/struct.wgsl: Added. * Source/WebGPU/WGSL/tests/valid/struct.wgsl: Added. Canonical link: https://commits.webkit.org/264066@main
02108de
to
65a2d2e
Compare
Committed 264066@main (65a2d2e): https://commits.webkit.org/264066@main Reviewed commits have been landed. Closing PR #13807 and removing active labels. |
65a2d2e
02108de