RFR: 8255150: Add utility methods to check long indexes and ranges [v2]

Dean Long dlong at openjdk.java.net
Fri Nov 6 04:27:57 UTC 2020


On Thu, 5 Nov 2020 23:58:21 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains ten commits:
>> 
>>  - Jorn's comments
>>  - Update headers and add intrinsic to Graal test ignore list
>>  - move compiler test and add bug to test
>>  - non x86_64 arch support
>>  - c2 test case
>>  - intrinsic
>>  - Use overloads of method names.
>>    
>>    Simplify internally to avoid overload resolution
>>    issues, leverging List<Number> for the exception
>>    mapper.
>>  - Vladimir's comments
>>  - checkLongIndex
>
> src/hotspot/share/opto/library_call.cpp line 1015:
> 
>> 1013:                   Deoptimization::Action_make_not_entrant);
>> 1014:   }
>> 1015: 
> 
> A comment here explaining what the code below is doing would be helpful.

This code wasn't here before, so I'm guessing it's needed for T_LONG.  For T_INT is it just wasted work?

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

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


More information about the core-libs-dev mailing list