RFR: 8367320: Sort cpu/x86 includes [v2]

[Francesco Andreuzzi]

On Wed, 10 Sep 2025 10:08:31 GMT, Francesco Andreuzzi <duke at openjdk.org> wrote:

>> src/hotspot/cpu/x86/continuationEntry_x86.inline.hpp line 29:
>> 
>>> 27: 
>>> 28: #include "oops/method.inline.hpp"
>>> 29: #include "runtime/continuationEntry.hpp"
>> 
>> I believe this is the "corresponding .hpp" in "All .inline.hpp files should include their corresponding .hpp file as the first include line..." from the style-guide, so should be kept at the top.
>
> I see. There's already a `continuationEntry_x86.hpp` in `cpu/x86`, maybe the `#include "runtime/continuationEntry.hpp"` should go there instead, and `continuationEntry_x86.inline.hpp` should include `continuationEntry_x86.hpp`? 
> 
> This would solve the problem without requiring changes in `SortIncludes.java`, which is not very flexible in terms of the relation between the names of the `.hpp` file and the corresponding `.inline.hpp` :
> 
> String nonInlineHpp = pathString.replace(".inline.hpp", ".hpp");

d7204d8f32399c9b437fbf83d9f66f068cc0a171

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27189#discussion_r2337020701