Bug 233891

Summary: Remove unneeded virtual allocator methods from Subspace.
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: ews-watchlist, keith_miller, msaboff, saam, tzagallo, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 232849    
Attachments:
Description Flags
proposed patch. ysuzuki: review+, ews-feeder: commit-queue-

Description Mark Lam 2021-12-06 12:27:39 PST 
Since the virtual allocate() and allocateFor() methods are now deleted, we can also rename the inline allocateNonVirtual() and allocatorForNonVirtual() methods to simply allocate() and allocateFor().  Similarly, rename allocatorForNonVirtualConcurrently() to allocatorForConcurrently().

There are 2 places that still invokes the non-inline version of CompleteSubspace::allocatorFor().  For this reason, we introduce a CompleteSubsace::allocatorForNonInline() to keep the linkage the same.  There's a chance that the compiler/linker may already inline the method in 1 or both of these places, but we'll offer allocatorForNonInline() to keep the code expressing the same thing and let the compiler/linker decide whether to inline it or not just as before.

This is purely a re-factoring patch.  There are no behavior changes, except for the removal of those 2 entries from the vtbls.
Comment 1 Radar WebKit Bug Importer 2021-12-06 12:28:32 PST 
<rdar://problem/86117970>
Comment 2 Mark Lam 2021-12-06 12:32:16 PST 
Created attachment 446068 [details]
proposed patch.
Comment 3 Yusuke Suzuki 2021-12-06 12:43:14 PST 
Comment on attachment 446068 [details]
proposed patch.

r=me
Comment 4 Mark Lam 2021-12-06 14:52:48 PST 
Thanks for the review.  Landed in r286572: <http://trac.webkit.org/r286572>.