Closed
Bug 513616
Opened 12 years ago
Closed 10 years ago
[meta] nanojit: clean up register allocator
Categories
(Core Graveyard :: Nanojit, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
(Whiteboard: Tracking)
Nanojit's register allocator is poorly documented, hard to understand, and contains at least one subtle-but-pervasive bug. I plan to improve this situation via a series of small patches that refactor code, improve documentation, and fix bugs. (In fact, I've already started doing this.) I plan to do it that way because I don't have a clear end goal in my mind, but I do see various small improvements to make, which well then enable further small improvements, etc. Lest this seems like needless pedantry and tidy-ups: making these changes will also improve my understanding of the register allocator in general, which will then help me make performance improvements to it later on. (The code improvements will also make it easier for others to make performance improvements.) This is a meta bug for tracking these improvements.
Comment 1•12 years ago
|
||
bug 514374 introduces an ugly use of allocator.free (~allocator.free). It should be cleaned up as part of this bug.
![]() |
Assignee | |
Comment 2•12 years ago
|
||
(In reply to comment #1) > bug 514374 introduces an ugly use of allocator.free (~allocator.free). It > should be cleaned up as part of this bug. I think we already had one occurrence of ~allocator.free anyway. Also, I don't think bug 514374 should be blocked by this meta-bug, as it will prevent bug 514374 from being closed until this bug is closed. Filing a new bug for removing ~allocator.free and making it block this bug would be better.
![]() |
Assignee | |
Comment 3•12 years ago
|
||
(In reply to comment #2) > > Also, I don't think bug 514374 should be blocked by this meta-bug, as it will > prevent bug 514374 from being closed until this bug is closed. Filing a new > bug for removing ~allocator.free and making it block this bug would be better. Done: bug 514551 created, and the 514374 block has been removed.
No longer blocks: 514374
Updated•11 years ago
|
Component: JavaScript Engine → Nanojit
QA Contact: general → nanojit
Whiteboard: Tracking
![]() |
Assignee | |
Comment 4•10 years ago
|
||
I'm gonna close this bug, it's served its purpose even though there are a couple of blocking bugs still open.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•