• Dominik Inführ's avatar
    Reland "[heap] Remove unused LocalSpace class" · c91c7270
    Dominik Inführ authored
    This is a reland of b0c70710
    
    The first CL got reverted because of build errors. This CL replaces the
    remaining usage of is_local_space() with is_compaction_space().
    Supposedly this was a leftover because https://crrev.com/c/2928189
    landed at roughly the same time.
    
    Original change's description:
    > [heap] Remove unused LocalSpace class
    >
    > LocalSpace was introduced for off-heap spaces with concurrent bytecode
    > compilation finalization. However, finalization ended up using
    > LocalHeap for concurrent allocations. LocalSpace is therefore unused
    > and can be removed.
    >
    > This CL removes LocalSpace and renames all mentions of local space to
    > compaction space. Compaction space was the only local space left.
    >
    > Change-Id: I12a8a2724f777a77ddb9957fe2d8e89febfebbaf
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2930169
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Commit-Queue: Dominik Inführ <dinfuehr@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#74914}
    
    Change-Id: I993c47fe85f4140f5d6137afde2653a48047cafb
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2939983Reviewed-by: 's avatarMichael Lippautz <mlippautz@chromium.org>
    Commit-Queue: Dominik Inführ <dinfuehr@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#74957}
    c91c7270
spaces-unittest.cc 12.1 KB