RFR: JDK-8315279: Factor 'basic_plus_adr' out of PhaseMacroExpand and delete make_load/store [v3]

Thu Sep 14 22:16:39 UTC 2023

On Thu, 31 Aug 2023 20:42:39 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Thanks for clarifying @iwanowww . I think I see your point now. My original intent was to just move these methods out of PhaseMacroExpand and not much else. 
>> 
>> I'm going to do some more refactoring and patch all users of these make methods to just use this single method: `static Node* make(PhaseIterGVN& igvn, Node* base, Node* ptr, Node* offset)`. What do you think?
>
> Sounds good. 
> 
> But in the future I'd like to see `PhaseMacroExpand` and `PhaseIdealLoop` migrated to `GraphKit` instead.

@iwanowww do you think I should just withdraw this PR and close the associated RFE?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15480#discussion_r1326567520