RFR: 8299274: Add elements to resolved_references consistently [v3]

Coleen Phillimore coleenp at openjdk.org
Thu Jan 5 19:17:50 UTC 2023


On Thu, 5 Jan 2023 14:58:20 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> I added a resolved_references_at and set_resolved_references_at() functions to use for accesses to the resolved_references array, except for the CDS usages. This has an assert that the element is an oop and oob checks and setting elements use a CAS consistently.
>> These asserts seem useful for finding the bug causing https://bugs.openjdk.org/browse/JDK-8296915.
>> Tested with tiers1-4.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Change the name to resolved_reference_at because there's a resolved_klass_at but I don't like this name as much as having resolved_references (plural) in the name.

Thanks for reviewing, Ioi.  singular resolved_reference looks better to me today than it did yesterday.

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

PR: https://git.openjdk.org/jdk/pull/11834


More information about the hotspot-dev mailing list