RFR: 8301996: Move field resolution information out of the cpCache [v2]
Martin Doerr
mdoerr at openjdk.org
Wed Jul 5 09:32:57 UTC 2023
On Thu, 29 Jun 2023 15:44:27 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:
>> 8301996: Move field resolution information out of the cpCache
>
> Matias Saavedra Silva has updated the pull request incrementally with one additional commit since the last revision:
>
> Coleen and Amit comments
src/hotspot/cpu/aarch64/templateTable_aarch64.cpp line 2538:
> 2536: load_resolved_field_entry(obj, cache, index, off, raw_flags, is_static);
> 2537: // Index holds the TOS
> 2538: __ mov(flags, index);
This is very confusing. You call the same thing "TOS", "type" and "index". Please use consistent naming and make it more comprehensive. In addition, why do you need the extra move?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14129#discussion_r1252826981
More information about the hotspot-dev
mailing list