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] Serialize global overrides #12860
[WGSL] Serialize global overrides #12860
Conversation
EWS run on previous version of this PR (hash 8a47a9e) |
return; | ||
|
||
m_stringBuilder.append("constant "); | ||
serializeVariable(variable); |
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.
Where is the validation that only certain types are allowed to be function constants?
Where is this tested?
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.
We have basically no validation of that sort yet. We could test by matching the metal output, but behavior tests won't work yet since we still need one more patch to get overrides to actually work.
8a47a9e
to
1c015db
Compare
EWS run on current version of this PR (hash 1c015db) |
https://bugs.webkit.org/show_bug.cgi?id=255600 rdar://108197601 Reviewed by Myles C. Maxfield. Add serialization support for override declarations. Notice that we temporarily disable the mangling of globals, which ironically wasn't being used yet, but requires notifying the API about the mangling. I will be implementing that next and will re-enable the mangling. * Source/WebGPU/WGSL/MangleNames.cpp: (WGSL::NameManglerVisitor::visit): * Source/WebGPU/WGSL/Metal/MetalFunctionWriter.cpp: (WGSL::Metal::FunctionDefinitionWriter::write): (WGSL::Metal::FunctionDefinitionWriter::visit): (WGSL::Metal::FunctionDefinitionWriter::visitGlobal): (WGSL::Metal::FunctionDefinitionWriter::serializeVariable): Canonical link: https://commits.webkit.org/263167@main
1c015db
to
e8be4ea
Compare
Committed 263167@main (e8be4ea): https://commits.webkit.org/263167@main Reviewed commits have been landed. Closing PR #12860 and removing active labels. |
e8be4ea
1c015db
π§ͺ gtk-wk2π§ͺ mac-wk2π§ͺ mac-AS-debug-wk2