Bug 21987 - CTI::putDoubleResultToJSNumberCellOrJSImmediate() hardcodes result register
Summary: CTI::putDoubleResultToJSNumberCellOrJSImmediate() hardcodes result register
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Cameron Zwarich (cpst)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-30 14:51 PDT by Cameron Zwarich (cpst)
Modified: 2008-10-30 15:17 PDT (History)
0 users

See Also:

Attachments
Proposed patch (1.18 KB, patch)
2008-10-30 15:00 PDT, Cameron Zwarich (cpst) 		oliver: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Cameron Zwarich (cpst) 2008-10-30 14:51:09 PDT 
CTI::putDoubleResultToJSNumberCellOrJSImmediate() hardcodes its result register as ecx, but it should clearly be tempReg1, which is currently ecx in all of its callsites.
Comment 1 Cameron Zwarich (cpst) 2008-10-30 15:00:05 PDT 
Created attachment 24783 [details]
Proposed patch
Comment 2 Cameron Zwarich (cpst) 2008-10-30 15:17:02 PDT 
Landed in r38012.