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

Vladimir Ivanov vlivanov at openjdk.org
Sat Sep 16 00:48:49 UTC 2023


On Thu, 14 Sep 2023 22:14:05 GMT, Cesar Soares Lucas <cslucas at openjdk.org> wrote:

>> 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?

It's up to you, Cesar. I find existing code good enough, but also I don't have anything against your proposal.

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

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


More information about the shenandoah-dev mailing list