Integrated: 8319439: Move BufferNode from PtrQueue files to new files

Kim Barrett kbarrett at openjdk.org
Wed Nov 15 01:11:52 UTC 2023


On Sat, 4 Nov 2023 11:25:05 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this simple (trivial?) change that moves the BufferNode class
> from share/gc/shared/ptrQueue.[ch]pp to new files
> share/gc/shared/bufferNode.[ch]pp.  The code (as opposed to the surrounding
> boiler plate and #include's) was moved unchanged.
> 
> This led to adding #include's of the new header file to a bunch of other files
> (include what you use), in some cases eliminating the need to #include
> ptrQueue.hpp.  Also tidied up the #include's for ptrQueue.[ch]pp to take
> advantage of the split.
> 
> Also, the gtest gc/shared/test_ptrQueueBufferAllocator.cpp is renamed to  
> gc/shared/test_bufferNodeAllocator.cpp, with some adjustments to includes and
> test naming.
> 
> Testing:
> mach5 tier1

This pull request has now been integrated.

Changeset: 1e76ba0c
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/1e76ba0cd0de38da6fc2b8147627496ee2f2837b
Stats:     383 lines in 17 files changed: 223 ins; 152 del; 8 mod

8319439: Move BufferNode from PtrQueue files to new files

Reviewed-by: tschatzl, iwalulya

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

PR: https://git.openjdk.org/jdk/pull/16506


More information about the hotspot-gc-dev mailing list