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

David Holmes dholmes at openjdk.org
Fri Jan 6 05:14:49 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.

> @dholmes-ora are you holding out until I change the name to something I don't agree with ?

No I was waiting to see if anyone else had an opinion on it. :)

Thanks for making the change.

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

Marked as reviewed by dholmes (Reviewer).

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


More information about the hotspot-dev mailing list