RFR: 8257498: Remove useless skeleton predicates [v5]

Mon Jan 25 10:41:45 UTC 2021

On Mon, 25 Jan 2021 10:13:27 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Christian Hagedorn has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
>> 
>>  - Merge master due to conflict
>>  - Revert get_skeleton_predicates again
>>  - Fix removal from global list and update get_skeleton_predicates
>>  - Add handling of Deoptimization::Reason_profile_predicate
>>  - 8257498: Remove useless skeleton predicates
>
> src/hotspot/share/opto/loopnode.cpp line 3597:
> 
>> 3595: 
>> 3596:   for (int i = C->skeleton_predicate_count(); i > 0; i--) {
>> 3597:     const int idx = i - 1;
> 
> Could be removed for consistency with above code.

Yes, that was a left over from a previous commit where we needed `idx` twice. I update that.

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

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