Closed
Bug 518034
Opened 16 years ago
Closed 16 years ago
NJ merge: minor LIR.cpp cleanups
Categories
(Tamarin Graveyard :: Baseline JIT (CodegenLIR), defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: n.nethercote, Assigned: rreitmai)
References
Details
Attachments
(1 file)
|
6.00 KB,
patch
|
Details | Diff | Splinter Review |
This patch:
- Has some whitespace changes.
- Wraps some uses of _stats.lir in verbose_only().
- Uses NULL instead of insImm(1) for a LIR_x condition (I'm not sure how TR works
because LirBufWriter::insGuard() asserts that the condition is NULL... hmm).
- Adds a missing comment.
|
Assignee | |
Comment 1•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 2•16 years ago
|
||
The 2 NanoAssert(0) in the 5th and 6th chunk of the patch case windows to error with 'unreachable code' please remove them on the tracemonkey side.
|
||
Comment 3•16 years ago
|
||
Resolved fixed engineering / work item that has been pushed. Setting status to verified.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•