RFR: 8366331: Sort share/prims includes [v3]

Francesco Andreuzzi duke at openjdk.org
Thu Aug 28 13:01:39 UTC 2025


On Thu, 28 Aug 2025 09:31:42 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   nn
>
> src/hotspot/share/prims/foreignGlobals.cpp line 25:
> 
>> 23: 
>> 24: #include "classfile/javaClasses.hpp"
>> 25: #include "foreignGlobals.hpp"
> 
> This does not look right, even in the original code. Isn't it the same as `prims/foreignGlobals.hpp`, which is subsumed by the existing `prims/foreignGlobals.inline.hpp`?

Right, removed in b0d163391270189772939ef2849e45dbf5963063

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26980#discussion_r2307338536


More information about the hotspot-dev mailing list