RFR: 8367656: Refactor Constantpool's operand array into two [v12]

Serguei Spitsyn sspitsyn at openjdk.org
Wed Oct 15 18:01:45 UTC 2025


On Wed, 15 Oct 2025 17:28:02 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> src/hotspot/share/oops/constantPool.inline.hpp line 90:
>> 
>>> 88:   return resolved_references()->obj_at(cache()->resolved_method_entry_at(index)->resolved_references_index());
>>> 89: }
>>> 90: 
>> 
>> Nit: Is this really needed?
>
> I believe that the style is to have an empty line between the code and the endif, so this is a style fix.

The issue is this is the only fix in this file. Should we go and fix all style issues around?
In fact, I'm okay with this tweak. :)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27198#discussion_r2433497456


More information about the serviceability-dev mailing list