[Webkit-unassigned] [Bug 87161] DFG CSE should do redundant store elimination

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=87161


Filip Pizlo <fpizlo at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #143387|0                           |1
        is obsolete|                            |
 Attachment #143470|                            |review?
               Flag|                            |




--- Comment #6 from Filip Pizlo <fpizlo at apple.com>  2012-05-22 23:12:35 PST ---
Created an attachment (id=143470)
 --> (https://bugs.webkit.org/attachment.cgi?id=143470&action=review)
the patch

Includes two fixes for handling implicit loads of local variables:

GetScopeChain reads the activation register. Hence, removing a SetLocal to the activation register that is live at the GetScopeChain is wrong.

There are other cases of implicit loads of locals, but they are already marked as either exiting or clobbering, so they are already handled correctly.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.