RESOLVED FIXED 162006
There is no good reason for WeakBlock to care about newly allocated objects 
https://bugs.webkit.org/show_bug.cgi?id=162006
Summary There is no good reason for WeakBlock to care about newly allocated objects
Filip Pizlo
Reported 2016-09-14 19:45:51 PDT
WeakBlock scans itself in two modes: visit: if a Weak in the block belongs to an unmarked object, ask the Weak to consider whether it should do things. reap: if a Weak in a block belongs to an unmarked object, delete the Weak. Except that "unmarked" has a peculiar meaning: WeakBlock defines it as !markedOrNewlyAllocated. So, a newly allocated object will never be consulted about anything. That sounds scary until you realize that newlyAllocated must have been cleared before we even got here. So, we were paying the price of checking newlyAllocated for no reason.
Attachments
possibly the patch (4.90 KB, patch)
2016-09-14 19:48 PDT, Filip Pizlo 		fpizlo: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2016-09-14 19:48:35 PDT
Created attachment 288919 [details] possibly the patch
Filip Pizlo
Comment 2 2016-09-15 10:17:45 PDT
Comment on attachment 288919 [details] possibly the patch r=ggaren in person
Filip Pizlo
Comment 3 2016-09-15 10:19:32 PDT
Landed in https://trac.webkit.org/changeset/205979
Note You need to log in before you can comment on or make changes to this bug.