RFR: 8372040: Remove Prefetch header vs inline header separation [v2]

Kim Barrett kbarrett at openjdk.org
Fri Jan 9 22:43:57 UTC 2026


On Fri, 9 Jan 2026 14:10:42 GMT, jonghoonpark <duke at openjdk.org> wrote:

>> related jira issue: https://bugs.openjdk.org/browse/JDK-8372040
>> 
>> ---
>> 
>> ## Changes
>> 
>> - Merged `prefetch.hpp` into `prefetch.inline.hpp` and removed the redundant `prefetch.hpp`. Updated all call sites accordingly.
>> - Removed `runtime/prefetch.inline.hpp` from `generation.hpp` to adhere to the rule that standard headers should not include inline headers.
>>     - Added the inclusion of `runtime/prefetch.inline.hpp` to `cardTableRS.cpp` instead.
>> - Removed `runtime/prefetch.hpp` from `g1YoungGCPostEvacuateTasks.cpp` as it is already included via `g1HeapRegion.inline.hpp`.
>> 
>> ---
>> 
>> Verified with tier1 testing; no regressions found.
>
> jonghoonpark has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add explicit prefetch include and clarify platform dependencies
>   
>   Signed-off-by: jonghoonpark <dev at jonghoonpark.com>

This PR (https://github.com/openjdk/jdk/pull/27189) (which was just closed as
inactive) discusses some issues similar to what came up here, about headers
that can't or shouldn't be included directly.

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

PR Comment: https://git.openjdk.org/jdk/pull/29096#issuecomment-3730823189


More information about the hotspot-dev mailing list