[Webkit-unassigned] [Bug 75342] Adjust spill order in DFG

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 4 22:18:30 PST 2012


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


Yuqiang Xian <yuqiang.xian at intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #120731|0                           |1
        is obsolete|                            |
 Attachment #120731|review?                     |
               Flag|                            |
 Attachment #121216|                            |review?
               Flag|                            |




--- Comment #7 from Yuqiang Xian <yuqiang.xian at intel.com>  2012-01-04 22:18:30 PST ---
Created an attachment (id=121216)
 --> (https://bugs.webkit.org/attachment.cgi?id=121216&action=review)
patch updated

Update the patch to _not_ adjust the spill order for 64bit, i.e. still Integers have lower priority to be spilled as there may be additional operations - in some cases the spilled integers are boxed as JS integers when being filled (btw, is it necessary?). The regression of the last patch on imaging-gaussian-blur should be caused by this.
So this patch keeps the spill order for 64bit as is, but simply modifies the comments a bit to better describe the reason, and also removes SpillOrderBoolean for 64bit as it's never used.

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



More information about the webkit-unassigned mailing list