Closed Bug 591914 Opened 14 years ago Closed 6 years ago

Remove large size branch from GC::Alloc

Categories

(Tamarin Graveyard :: Garbage Collection (mmGC), defect)

Product:
Tamarin Graveyard
Component:
Garbage Collection (mmGC)
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX
Milestone:
Future

People

(Reporter: treilly, Unassigned)

References

Details

(Whiteboard: PACMAN)

If we made GCLargeAlloc a GCAlloc and placed it at the end of the GCAlloc's array and had a branch-free way to clamp the size class index to the end of the size class array we could remove a branch from GC::Alloc I think this can be made to work: http://bob.allegronetwork.com/prog/tricks.html#clamp
Whiteboard: PACMAN
Target Milestone: --- → Future
This is probably not important because in almost all cases that test is or can be constant-folded (needs to be substantiated, but it is credible).
Flags: flashplayer-qrb+
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.