Closed
Bug 519575
Opened 15 years ago
Closed 15 years ago
NJ merge: more LIR.h drift
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: graydon, Assigned: graydon)
References
Details
(Whiteboard: fixed-in-tracemonkey)
Attachments
(1 file)
|
7.19 KB,
patch
|
gal
:
review+
|
Details | Diff | Splinter Review |
This patch pulls in a couple harmless LIR.h hunks and removes a dead ctor from Assembler. Only one of the hunks is interesting: the one that moves the 'protected:' visibility line in LirBufWriter to hide the maximum skip size.
I went through jstracer.cpp and removed all the uses of that skip size. They're all irrelevant now anyway, since none of the allocations route through to skip, and they all look to be capped by things like "stack depth" already anyway if we're concerned about user data getting a new avenue to chew up allocation.
Attachment #403631 -
Flags: review?(gal)
|
||
Updated•15 years ago
|
Attachment #403631 -
Flags: review?(gal) → review+
|
Assignee | |
Comment 1•15 years ago
|
||
Whiteboard: fixed-in-tracemonkey
|
||
Comment 2•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•15 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•