RFR: 8369002: Extract the loop->is_member(get_loop(get_ctrl(node))) pattern in a new function [v3]
Benoît Maillard
bmaillard at openjdk.org
Thu Nov 13 13:35:51 UTC 2025
On Thu, 13 Nov 2025 10:36:25 GMT, Anton Seoane Ampudia <aseoane at openjdk.org> wrote:
>> This PR adds a "shorthand" for the common `loop->is_member(get_loop(get_ctrl(node)))` pattern in loop optimizations.
>>
>> In PhaseIdealLoop, there is already an `is_member` function that checks if a node is a (nested) member of an IdealLoopTree. In a similar fashion, this changeset adds a `ctrl_is_member` that aims to simplify the common pattern of:
>>
>> Node* node_ctrl = get_ctrl(node);
>> if (loop->is_member(get_loop(node))) { ... }
>>
>>
>> This hopes to provide a bit more readability and code conciseness in such a common operation.
>>
>> **Testing:** passes tiers 1-3
>
> Anton Seoane Ampudia has updated the pull request incrementally with one additional commit since the last revision:
>
> Review comments: nit
Marked as reviewed by bmaillard (Committer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/28259#pullrequestreview-3459822128
More information about the hotspot-compiler-dev
mailing list