On Tue, 10 May 2022 20:53:37 GMT, Vladimir Ivanov <vliva...@openjdk.org> wrote:

>> src/hotspot/share/utilities/growableArray.hpp line 151:
>> 
>>> 149:     return _data;
>>> 150:   }
>>> 151: 
>> 
>> This accessor is added to be able to temporarily view a stable GrowableArray 
>> instance as a C-style array. It is used to by `NativeCallConv` and 
>> `RegSpiller` in `foreign_globals.hpp`.
>> 
>> GrowableArray already has an `adr_at` accessor that does something similar, 
>> but using `adr_at(0)` fails on empty growable arrays since it also performs 
>> a bounds check. So it can not be used.
>
> Any problems with migrating `CallConv` and `RegSpiller`away from ` VMReg* + 
> int` to `GrowableArray<VMReg>`?

I'll try migrating to `GrowableArray`

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

PR: https://git.openjdk.java.net/jdk/pull/7959

Reply via email to