[webkit-reviews] review requested: [Bug 235382] [JSC] move function wrapping logic to a new Function type : [Attachment 449517] Patch

[bugzilla-daemon at webkit.org]

Caitlin Potter (:caitp) <caitp at igalia.com> has asked  for review:
Bug 235382: [JSC] move function wrapping logic to a new Function type
https://bugs.webkit.org/show_bug.cgi?id=235382

Attachment 449517: Patch

https://bugs.webkit.org/attachment.cgi?id=449517&action=review




--- Comment #1 from Caitlin Potter (:caitp) <caitp at igalia.com> ---
Created attachment 449517

  --> https://bugs.webkit.org/attachment.cgi?id=449517&action=review

Patch

This is just a starter patch, it has some things which could be better --- For
the thunk, the rethrowing logic does not actually work, and I need some
clarification on getting that set up. It might be good for allocation to occur
inline, rather than in C operations. As noted in the changelog, at this time
thhtere isn't anything added to DFG/FTL, but potentially there are
opportunities there. I'm hoping I can get some feedback on the initial version
and see if we actually prefer this to the JS builtin we've already got.