This is only used in the Checker, and it can easily keep track of the current function. Doing it this way spares us from tracking the current function in the name resolver, and saves 8 bytes from every Return.
Created attachment 374046 [details] Patch Not yet up for review because I've not yet tested it.
Comment on attachment 374046 [details] Patch It passes all tests, so asking for review.
Comment on attachment 374046 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=374046&action=review > Source/WebCore/Modules/webgpu/WHLSL/AST/WHLSLReturn.h:-60 > - void setFunction(FunctionDefinition* functionDefinition) { m_function = functionDefinition; } We might have to add this back in if we add a pass to inline things.
Comment on attachment 374046 [details] Patch Clearing flags on attachment: 374046 Committed r247419: <https://trac.webkit.org/changeset/247419>
All reviewed patches have been landed. Closing bug.
<rdar://problem/53054937>