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

Cesar Soares Lucas cslucas at openjdk.org
Wed Aug 30 03:20:38 UTC 2023


I believe the factory methods for AddPNode should be in the AddPNode class. The make_load / make_store methods in PhaseMacroExpand can be refactored to instead just use the "make" methods from Load/Store classes.

Tested with tier1-3.

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

Commit messages:
 - Refactoring on PhaseMacroExpand

Changes: https://git.openjdk.org/jdk/pull/15480/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15480&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8315279
  Stats: 92 lines in 11 files changed: 37 ins; 32 del; 23 mod
  Patch: https://git.openjdk.org/jdk/pull/15480.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15480/head:pull/15480

PR: https://git.openjdk.org/jdk/pull/15480


More information about the shenandoah-dev mailing list