RFR: 8365765: thread.inline.hpp includes the wrong primary header file [v2]

Stefan Karlsson stefank at openjdk.org
Fri Aug 22 09:55:51 UTC 2025


On Fri, 22 Aug 2025 09:44:43 GMT, Paul Hübner <duke at openjdk.org> wrote:

>> Hi all,
>> 
>> This ensures `thread.inline.hpp` adheres to the style guide:
>>> All .inline.hpp files should include their corresponding .hpp file as the first include line with a blank line separating it from the rest of the include lines. [...]
>> 
>> I've run tests with JDK tiers 1-3 on macOS (AArch64, x64), Linux (AArch64, x64), and Windows (x64); all green.
>
> Paul Hübner has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update src/hotspot/share/runtime/thread.inline.hpp
>   
>   Co-authored-by: Stefan Karlsson <stefan.karlsson at oracle.com>

Looks good!

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

Marked as reviewed by stefank (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26896#pullrequestreview-3143866030


More information about the hotspot-runtime-dev mailing list