RFR: 8315954: getArgumentValues002.java fails on Graal [v2]

[Tom Rodriguez]

On Wed, 13 Sep 2023 09:47:48 GMT, Doug Simon <dnsimon at openjdk.org> wrote:

>> Doug Simon has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - generalized getLiveObjectLocalsAt to getOopMapAt
>>  - need to zero oop_map_buf
>>  - simplified getLiveObjectLocalsAt and moved it from ResolvedJavaMethod to HotSpotResolvedJavaMethod
>
> src/hotspot/share/interpreter/oopMapCache.cpp line 204:
> 
>> 202: void InterpreterOopMap::initialize() {
>> 203:   _method    = nullptr;
>> 204:   _mask_size = INT_MAX;  // This value should cause a failure quickly
> 
> Unless I'm mistaken, `USHRT_MAX` is a legal (but unlikely) value (i.e. `max_locals` in a class file can be 65635) so I changed this to use `INT_MAX` instead.

It's actually not legal since _mask_size is always a multiple of `bits_per_entry` which is 2 so any odd value would work.  I would be leery of touching this unless you really need to.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15705#discussion_r1324817483