On Wed, 30 Apr 2025 15:33:52 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> This PR is based on the work of @mernst-github and aims to implement an 
>> _internal_ thread-local 'stack' allocator, which works like a dynamically 
>> sized arena, but with reset functionality to reset the allocated size back 
>> to a certain level. The underlying memory could stay around between calls, 
>> which could improve performance.
>> 
>> Re-allocated segments are not zeroed between allocations.
>
> Per Minborg has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Improve on comments

src/java.base/share/classes/jdk/internal/foreign/BufferStack.java line 108:

> 106:             final Arena arena = Arena.ofAuto();
> 107:             final SlicingAllocator stack = new 
> SlicingAllocator(arena.allocate(size));
> 108:             return new PerThread(new ReentrantLock(), stack, arena);

Suggestion:

            final Arena arena = Arena.ofAuto();
            return new PerThread(new ReentrantLock(), 
                                 new SlicingAllocator(arena.allocate(size)), 
arena);
        }

The local variable stack is only used once, is it okay not to use it?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24829#discussion_r2071260412

Reply via email to