RFR: 8335257: Refactor code to create Initialized Assertion Predicates into separate class [v2]
Emanuel Peter
epeter at openjdk.org
Fri Jul 5 09:08:20 UTC 2024
On Fri, 5 Jul 2024 09:00:49 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> 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:
>>
>> - review Emanuel
>> - Merge branch 'refs/heads/master' into JDK-8335257
>> - 8335257: Refactor code to create Initialized Assertion Predicates into separate class
>
> src/hotspot/share/opto/predicates.cpp line 427:
>
>> 425: assertion_expression.clone_and_replace_init_and_stride(_new_init, _new_stride, control, _phase);
>> 426: OpaqueInitializedAssertionPredicateNode* assertion_predicate_expression
>> 427: = new OpaqueInitializedAssertionPredicateNode(tmp_opaque->in(1)->as_Bool(), _phase->C);
>
> Decide if you want the `=` on the first or last line and do it consistently ;)
Long names are so nice, but the code does get a big difficult to format 😅
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19940#discussion_r1666545084
More information about the hotspot-compiler-dev
mailing list