Closed
Bug 518906
Opened 16 years ago
Closed 16 years ago
NJ merge: bring fragment profiling to TR
Categories
(Tamarin Graveyard :: Baseline JIT (CodegenLIR), defect)
Tamarin Graveyard
Baseline JIT (CodegenLIR)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: graydon, Assigned: rreitmai)
References
Details
Attachments
(1 file, 3 obsolete files)
|
15.85 KB,
patch
|
Details | Diff | Splinter Review |
TM has a per-fragment profiling facility, introduced in bug 503424, that TR lacks. This patch adds it.
As far as I know you're not using your GuardRecords or SideExits, but we are, and the profiling code uses them. So I moved them from core/Verifier.h to nanojit/Native.h.
|
Reporter | |
Comment 1•16 years ago
|
||
Of course, I didn't test the former patch under --enable-debug, which is what turns it *on*. This patch goes the remainder of the ways.
Attachment #402901 -
Attachment is obsolete: true
|
|
Reporter | |
Comment 2•16 years ago
|
||
Just a refresh, still contains the same stuff.
Attachment #402952 -
Attachment is obsolete: true
|
Assignee | |
Comment 3•16 years ago
|
||
rebase and minor changes based on bug 519561 and 519527 landing first
Attachment #403580 -
Attachment is obsolete: true
Attachment #405126 -
Flags: superreview?(edwsmith)
|
|
Assignee | |
Comment 4•16 years ago
|
||
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
|
||
Comment 5•16 years ago
|
||
Resolved fixed engineering / work item that has been pushed. Setting status to verified.
Status: RESOLVED → VERIFIED
|
|
Assignee | |
Updated•16 years ago
|
Attachment #405126 -
Flags: superreview?(edwsmith)
You need to log in
before you can comment on or make changes to this bug.
Description
•