8372040: Remove Prefetch header vs inline header separation - Inquiry regarding contributing

[David Holmes]

On 6/01/2026 3:43 am, 박종훈 wrote:
> Hello,
> 
> I would like to contribute to the issue JDK-8372040: Remove Prefetch 
> header vs inline header separation (https://bugs.openjdk.org/browse/ 
> JDK-8372040 <https://bugs.openjdk.org/browse/JDK-8372040>).
> 
> Following the suggestion in the issue description, I have implemented 
> the changes as follows:
> 
> 1. Merged `prefetch.hpp` into `prefetch.inline.hpp`.
> 
> 2. Removed the original `prefetch.hpp`.
> 
> 3. Updated platform-specific `prefetch_<arch>.inline.hpp` files to 
> include `prefetch.inline.hpp` instead of `prefetch.hpp`.
> 
> 4. Updated **`g1YoungGCPostEvacuateTasks.cpp`** to include 
> `prefetch.inline.hpp`, replacing its current inclusion of the now- 
> removed `prefetch.hpp`.
> 
> I have run the tier1 tests and no new regressions were observed.
> 
> Would it be okay to proceed with this approach?

That sounds reasonable, but you also need to change 
./share/gc/serial/generation.hpp so that it doesn't include the 
inline.hpp file, and then adjust the serial .cpp files that actually 
need the prefetch functions.

Thanks,
David
> Best regards, Jonghoon Park