Closed Bug 511744 Opened 16 years ago Closed 14 years ago

TM: Replace Queue<> with JSTempVector<>

Categories

(Core :: JavaScript Engine, defect)

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

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: graydon, Assigned: graydon)

References

Details

Queue<> is a bit hokey, and it's inefficient as it always hits malloc (when not hitting nanojit::Allocator, anyway). It'd be better to use JSTempVector<>. For this to work, JSTempVector<> needs to be (re-)parameterized with an allocator policy, so it can take a nanojit::Allocator& to use in appropriate cases. Possibly also a parameter describing its error-handling policy. Apparently this already exists for JSHashMap / JSPool, over in bug 506410. Perhaps a new bug this one depends on for policy-izing JSTempVector<>?
Depends on: 511750
Obsolete with the removal of tracejit.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.