RFR: 8299274: Add elements to resolved_references consistently [v3]
Ioi Lam
iklam at openjdk.org
Thu Jan 5 17:28:50 UTC 2023
On Wed, 4 Jan 2023 15:11:30 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>>> I need it to return the existing value for the code in resolve_constant_at_impl().
>>
>> So it's a getter/setter, but if the argument must be nullptr, I think we should skip it.
>
> obj_at_put already checks the index.
I agree with David's naming.
For grepping, it's better to grep for the singular form as there are already functions that use the singular form.
$ find src/hotspot -type f | xargs grep resolved_reference | grep -v references | wc
41 223 4559
Here are some examples. If you grep for the plural form, you will miss them.
./share/oops/constantPool.cpp: set_resolved_reference_length(
./share/oops/constantPool.hpp: int _resolved_reference_length;
./share/oops/cpCache.hpp: void initialize_resolved_reference_index(int ref_index) {
-------------
PR: https://git.openjdk.org/jdk/pull/11834
More information about the hotspot-dev
mailing list