Bug 78392 - Update MIPS patchOffsetGetByIdSlowCaseCall
Summary: Update MIPS patchOffsetGetByIdSlowCaseCall
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-10 15:19 PST by Chao-ying Fu
Modified: 2012-02-27 18:13 PST (History)
3 users (show)

See Also:

Attachments
Update patchOffsetGetByIdSlowCaseCall values (1.54 KB, patch)
2012-02-10 15:27 PST, Chao-ying Fu 		no flags Details | Formatted Diff | Diff
Update patchOffsetGetByIdSlowCaseCall values (1.53 KB, patch)
2012-02-27 18:11 PST, Chao-ying Fu 		barraclough: review+
barraclough: commit-queue+
Details | Formatted Diff | Diff
Show Obsolete (1) 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 Chao-ying Fu 2012-02-10 15:19:17 PST 
Hi,
Due to recent change, I need to update MIPS patchOffsetGetByIdSlowCaseCall values.  I will post a patch soon.  Thanks!

Regards,
Chao-ying
Comment 1 Chao-ying Fu 2012-02-10 15:27:39 PST 
Created attachment 126592 [details]
Update patchOffsetGetByIdSlowCaseCall values

Here is the patch.  Testing is ok.  Thanks!
Comment 2 Eric Seidel (no email) 2012-02-13 11:39:50 PST 
Who should review this?  I believe this only affects MIPS?  How do we test this change?
Comment 3 WebKit Review Bot 2012-02-13 12:32:04 PST 
Comment on attachment 126592 [details]
Update patchOffsetGetByIdSlowCaseCall values

Clearing flags on attachment: 126592

Committed r107594: <http://trac.webkit.org/changeset/107594>
Comment 4 WebKit Review Bot 2012-02-13 12:32:15 PST 
All reviewed patches have been landed.  Closing bug.
Comment 5 Chao-ying Fu 2012-02-27 18:11:22 PST 
Created attachment 129155 [details]
Update patchOffsetGetByIdSlowCaseCall values

The MIPS patchOffsetGetByIdSlowCaseCall values are wrong again, due to recent change in updateTopCallFrame().
Here is the patch to corret them.  JIT testing is ok.
Thanks a lot!

Regards,
Chao-ying