Closed Bug 580529 Opened 14 years ago Closed 14 years ago

Running sunspider through eval leads to significant slowdowns in some cases.

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: paul.biggar, Unassigned)

References

Details

(Keywords: meta)

Attachments

(1 file)

Make sunspider use eval instead of load (diff against sunspider svn rev 63802)
810 bytes, patch
Details | Diff | Splinter Review 
When running sunspider using eval (see patch), several benchmarks take signficantly more time:

3d-morph: 52ms vs 20ms
bitops-0bitwise-and: 395ms vs 3ms
date-format-xparb: 64ms vs 30ms
string-base-64: 21ms vs 11 ms

I've confirmed with gal that this shouldn't happen.
Attachment #458919 - Attachment filename: a → eval.patch
Attachment #458919 - Attachment is patch: true
Attachment #458919 - Attachment mime type: application/octet-stream → text/plain
Dup of bug 489098 ?

/be
Whiteboard: DUPEME
Let's make this meta, then.

Paul, want to take assignment of bug 489098 ? Gal and I can help.

/be
Depends on: 489098
Keywords: meta
Whiteboard: DUPEME
OK.
Bug 489098 is fixed-in-tracemonkey, and I verified that the slowdown here is resolved.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: