[webkit-reviews] review granted: [Bug 125881] frameRegisterCount() should include maxFrameExtentForSlowPathCall. : [Attachment 219479] patch 2: applied requested fixes.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Dec 17 17:43:35 PST 2013


Geoffrey Garen <ggaren at apple.com> has granted Mark Lam <mark.lam at apple.com>'s
request for review:
Bug 125881: frameRegisterCount() should include maxFrameExtentForSlowPathCall.
https://bugs.webkit.org/show_bug.cgi?id=125881

Attachment 219479: patch 2: applied requested fixes.
https://bugs.webkit.org/attachment.cgi?id=219479&action=review

------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
Let's remove the JSStack::CallerFrameAndPCSize line. The slow path helper, by
definition, must include those. (That value seems to have been added in
<http://trac.webkit.org/changeset/160506> as a crude estimate of
maxFrameExtentForSlowPathCallInRegisters.)


More information about the webkit-reviews mailing list