Bug 139827 - Stop pretending that the LexicalEnvironment local var is captured
Summary: Stop pretending that the LexicalEnvironment local var is captured
Status: RESOLVED DUPLICATE of bug 141174
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mark Lam
Keywords: InRadar
Depends on: 140093 140097
Blocks: 139524
  Show dependency treegraph
Reported: 2014-12-19 12:12 PST by Mark Lam
Modified: 2015-04-13 13:02 PDT (History)
2 users (show)

See Also:

the patch. (1.38 KB, patch)
2015-01-05 12:40 PST, Mark Lam
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Lam 2014-12-19 12:12:11 PST
The current code pretends that the LexicalEnvironment local var is captured.  This is a short cut to keeping that local var alive (even when it doesn't need to be).  However, this pretense does have negative side-effects (see https://bugs.webkit.org/show_bug.cgi?id=139524).  So, let's fix it.
Comment 1 Radar WebKit Bug Importer 2014-12-19 12:14:35 PST
Comment 2 Mark Lam 2015-01-05 12:40:52 PST
Created attachment 243988 [details]
the patch.

This patch is just to track the work being done.  It is not ready for landing because applying this patch will expose other issues manifesting as esc-stress-test failures.  Bugs tracking those issues will be documented shortly.
Comment 3 Mark Lam 2015-01-05 12:42:19 PST
(In reply to comment #2)
> ... esc-stress-test failures. ...

typo: /esc-stress-test/jsc-stress-test/
Comment 4 Mark Lam 2015-04-13 13:02:42 PDT
The offending code was deleted when CodeBlock::isCaptured() was deleted as part of the fix to https://bugs.webkit.org/show_bug.cgi?id=141174.  So, I'll resolve this as a dup to 141174.

*** This bug has been marked as a duplicate of bug 141174 ***