[webkit-reviews] review granted: [Bug 200429] Mangled WHLSL names don't need to allocate Strings : [Attachment 375508] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun Aug 4 18:15:36 PDT 2019
Saam Barati <sbarati at apple.com> has granted Sam Weinig <sam at webkit.org>'s
request for review:
Bug 200429: Mangled WHLSL names don't need to allocate Strings
https://bugs.webkit.org/show_bug.cgi?id=200429
Attachment 375508: Patch
https://bugs.webkit.org/attachment.cgi?id=375508&action=review
--- Comment #3 from Saam Barati <sbarati at apple.com> ---
Comment on attachment 375508
--> https://bugs.webkit.org/attachment.cgi?id=375508
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=375508&action=review
r=me
Just curious: Did you measure any compile time speedup? I like to flip the
“dumpPhaseTimes” Boolean to true in WHLSLPrepare and run some shaders to
measure compile time changes. I’ll email you the web page I use.
> Source/WebCore/Modules/webgpu/WHLSL/Metal/WHLSLMangledNames.h:159
> + return string.is8Bit();
Why not an assert?
More information about the webkit-reviews
mailing list