RFR: 8331168: Introduce PredicateEntryIterator to iterate through predicate entries [v2]
Roland Westrelin
roland at openjdk.org
Tue Jun 18 12:09:18 UTC 2024
On Mon, 17 Jun 2024 11:00:59 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> This is another small PR split off from the complete fix for Assertion Predicates.
>>
>> Changes include:
>> - Replace predicate matching/skipping code in `build_loop_late_post_work()` to use dedicated walker (i.e. new `PredicateEntryIterator` class) which uses predicate classes.
>> - New predicate helper classes to improve readability.
>> - Simple typos and code style updates.
>>
>> Thanks,
>> Christian
>
> Christian Hagedorn has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>
> - Introduce is_strict_dominator()
> - Merge branch 'master' into JDK-8331168
> - 8331168: Introduce PredicateEntryIterator to iterate through predicate entries
Looks good to me.
-------------
Marked as reviewed by roland (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19716#pullrequestreview-2125338016
More information about the hotspot-compiler-dev
mailing list