Bug 4419

Summary:

speed up JavaScript by improving KJS::List

Product:

WebKit

Reporter:

Darin Adler <darin>

Component:

JavaScriptCore

Assignee:

Darin Adler <darin>

Status:

RESOLVED FIXED

Severity:

Enhancement

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
patch that makes delete inline and makes list pool entries 12 bytes smaller	none
patch that makes delete inline and makes list pool entries 12 bytes smaller	none
improved patch that deletes an empty function left behind before	mjs: review+

Darin Adler

Reported 2005-08-13 22:56:09 PDT

I did some analysis of JavaScript iBench performance, and found some speedups. This one is additional inlining of KJS::List functions.

Attachments
patch that makes delete inline and makes list pool entries 12 bytes smaller (3.83 KB, patch) 2005-08-13 22:57 PDT, Darin Adler	no flags	Details Formatted Diff Diff
patch that makes delete inline and makes list pool entries 12 bytes smaller (3.83 KB, patch) 2005-08-13 22:57 PDT, Darin Adler	no flags	Details Formatted Diff Diff
improved patch that deletes an empty function left behind before (4.18 KB, patch) 2005-08-13 23:38 PDT, Darin Adler	mjs: review+	Details Formatted Diff Diff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.

Darin Adler

Comment 1 2005-08-13 22:57:21 PDT

Created attachment 3366 [details] patch that makes delete inline and makes list pool entries 12 bytes smaller

Darin Adler

Comment 2 2005-08-13 22:57:22 PDT

Created attachment 3367 [details] patch that makes delete inline and makes list pool entries 12 bytes smaller

Darin Adler

Comment 3 2005-08-13 23:05:32 PDT

I measured a 1% improvement on JavaScript iBench performance on my machine with this patch.

Darin Adler

Comment 4 2005-08-13 23:38:26 PDT

Created attachment 3371 [details] improved patch that deletes an empty function left behind before

Note You need to log in before you can comment on or make changes to this bug.