Patch forthcoming.
Created attachment 280002 [details] the patch
Comment on attachment 280002 [details] the patch r=me
Landed in http://trac.webkit.org/changeset/201470
Comment on attachment 280002 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=280002&action=review > Source/JavaScriptCore/ChangeLog:9 > + value, presumably because this makes it soooper fast. In DFG IR, empty values can appear I'm going to run the experiment where jsDynamicCast<Foo*>(JSValue()) returns nullptr and see what the performance is like. Do you know if anyone has measured the performance impact of this recently?
(In reply to comment #4) > Comment on attachment 280002 [details] > the patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=280002&action=review > > > Source/JavaScriptCore/ChangeLog:9 > > + value, presumably because this makes it soooper fast. In DFG IR, empty values can appear > > I'm going to run the experiment where jsDynamicCast<Foo*>(JSValue()) returns > nullptr and see what the performance is like. > Do you know if anyone has measured the performance impact of this recently? I don't think anyone has ever measured it!