On Thu, 5 Nov 2020 23:58:21 GMT, Dean Long <dl...@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
