RFR: 8313435: Clean up unused default methods code [v2]

David Holmes dholmes at openjdk.org
Thu Aug 3 05:07:34 UTC 2023


On Wed, 2 Aug 2023 12:18:27 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/classfile/defaultMethods.cpp line 1081:
>> 
>>> 1079:   int new_idx = 0;
>>> 1080: 
>>> 1081:   for (u2 i = 0; i < new_size; ++i) {
>> 
>> Is this change for the loop index necessary? Even as an int it is constrained to be <= max(u2) - and `at_put` expects an int index doesn't it?
>
> Later it uses 'i' to be method_idnum, which is constrained to be u2.

okay

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15095#discussion_r1282646713


More information about the hotspot-runtime-dev mailing list