RFR: 8378793: Add ResolvedFieldEntry is_valid assert [v2]
David Simms
dsimms at openjdk.org
Fri Feb 27 14:19:19 UTC 2026
On Fri, 27 Feb 2026 13:52:45 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> Please review this small change to add an assert for values in ResolvedFieldEntry, which was useful for debugging at one time.
>> Tested with tier1-3
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>
> Make assert_is_valid() only debug and only on ResolvedFieldEntry initialization. Change fill_in to initialize flags also from fieldDescriptor, so set_flags and set_bytecodes can be private.
LGTM. It might feel like overkill, but during Valhalla modifications for get/putfield lead to using a lot of registers, and often bugs were introduced that testing took a really long time to discover compared to "assertion overkill" (they are debug only, basically free).
-------------
Marked as reviewed by dsimms (Author).
PR Review: https://git.openjdk.org/jdk/pull/29943#pullrequestreview-3867172475
More information about the hotspot-dev
mailing list