Closed
Bug 412308
Opened 18 years ago
Closed 18 years ago
TT: early and optimistic binding changelist 273 causes dramatic slowdown in AS3Chess benchmark
Categories
(Tamarin Graveyard :: Virtual Machine, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: stejohns, Assigned: edwsmith)
References
Details
Attachments
(1 file)
|
9.53 KB,
patch
|
rreitmai
:
review+
stejohns
:
review+
|
Details | Diff | Splinter Review |
I was getting nps>200 before, now I get nps ~40. Dropping MAXTRACE from 20000 back to 2000 helps some (nps ~160) but clearly something odd is going on. Needs investigating.
|
Assignee | |
Comment 1•18 years ago
|
||
I'm seeing huge amounts of time in mark/sweep, as well as many trace buffer flushes. Investigating whether calling LCompressedBuffer.clear() after each trace is generated will let us avoid calling GC::Collect() explicitly.
|
|
Assignee | |
Comment 2•18 years ago
|
||
Also
- support SWITCH in trace()
- free LIR buffers explicitly at end-of-trace time
Attachment #297228 -
Flags: review?
|
|
Assignee | |
Updated•18 years ago
|
Attachment #297228 -
Flags: review?(stejohns)
Attachment #297228 -
Flags: review?(rreitmai)
Attachment #297228 -
Flags: review?
|
|
Reporter | |
Updated•18 years ago
|
Attachment #297228 -
Flags: review?(stejohns) → review+
|
|
||
Comment 4•18 years ago
|
||
Comment on attachment 297228 [details] [diff] [review]
Bump trace cache to 16M to avoid thrashing
we might need to tweak pendingTrace.gen later since it holds onto the buffer. But that should be fine for now until we implement trees.
Attachment #297228 -
Flags: review?(rreitmai) → review+
|
|
Assignee | |
Comment 5•18 years ago
|
||
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
|
|
||
Comment 6•16 years ago
|
||
Closing out all TT: transfer bugs that are resolved fixed.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•