...
Doing this next.
Created attachment 370375 [details] WIP contains all the dumper code. Will rebase once that lands.
Created attachment 370380 [details] patch
Comment on attachment 370380 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=370380&action=review > LayoutTests/webgpu/whlsl-dont-crash-parsing-enum.html:1 > +<!DOCTYPE html> Will try to reduce this test case to just run the WHLSL parser. But I don't have a build yet to get against. Will fix for final patch.
Comment on attachment 370380 [details] patch Thanks for catching this! Is this the only place in the file where this bad pattern occurs?
(In reply to Myles C. Maxfield from comment #5) > Comment on attachment 370380 [details] > patch > > Thanks for catching this! Is this the only place in the file where this bad > pattern occurs? Which file are we talking about here? (Maybe we should make the name() method not return "String&").
Comment on attachment 370380 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=370380&action=review >> LayoutTests/webgpu/whlsl-dont-crash-parsing-enum.html:1 >> +<!DOCTYPE html> > > Will try to reduce this test case to just run the WHLSL parser. But I don't have a build yet to get against. Will fix for final patch. gonna just stick with this.
Created attachment 370466 [details] patch for landing
Comment on attachment 370466 [details] patch for landing Clearing flags on attachment: 370466 Committed r245662: <https://trac.webkit.org/changeset/245662>
All reviewed patches have been landed. Closing bug.
<rdar://problem/51049758>