[lworld] RFR: 8377242: [lworld] Remove _has_inlined_fields member from ClassFileParser
Frederic Parain
fparain at openjdk.org
Thu Feb 5 21:36:16 UTC 2026
On Thu, 5 Feb 2026 13:07:18 GMT, Casper Norrbin <cnorrbin at openjdk.org> wrote:
> Hi everyone,
>
> This PR removes the redundant `ClassFileParser::_has_inlined_fields` member from ClassFileParser. The value stored in `_has_inlined_fields` is derived directly from `FieldLayoutInfo` and is only used for a getter. Since `ClassFileParser` already retains this in `_layout_info`, a separate duplicate member is unnecessary. With this change, the getter returns the value directly from the stored `FieldLayoutInfo`, removing the extra member while preserving existing behavior.
>
> Testing:
> - Tier 1
LGTM
-------------
Marked as reviewed by fparain (Committer).
PR Review: https://git.openjdk.org/valhalla/pull/2047#pullrequestreview-3759571642
More information about the valhalla-dev
mailing list