[webkit-reviews] review granted: [Bug 122065] Get rid of the AlreadyInJSStack recoveries since they are totally redundant with the DisplacedInJSStack recoveries : [Attachment 213022] the patch

[bugzilla-daemon at webkit.org]

Mark Hahnenberg <mhahnenberg at apple.com> has granted Filip Pizlo
<fpizlo at apple.com>'s request for review:
Bug 122065: Get rid of the AlreadyInJSStack recoveries since they are totally
redundant with the DisplacedInJSStack recoveries
https://bugs.webkit.org/show_bug.cgi?id=122065

Attachment 213022: the patch
https://bugs.webkit.org/attachment.cgi?id=213022&action=review

------- Additional Comments from Mark Hahnenberg <mhahnenberg at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=213022&action=review


r=me + style bot stuff

> Source/JavaScriptCore/ChangeLog:18
> +	   after the forward exit are MovHints or other tnings that the OSR
exit compiler can

s/tnings/things

> Source/JavaScriptCore/ChangeLog:30
> +	   speculation direction of the node for which it is doing a
conversion. Since SetLocal's

s/SetLocal's/SetLocals