RFR: 8299191: Unnecessarily global friend functions for relocInfo [v2]

Kim Barrett kbarrett at openjdk.org
Thu Dec 22 17:35:16 UTC 2022


> Please review this small cleanup around the relocInfo class.  It declares a
> couple of global functions as friends, so they have access to private
> constructors and helper functions.  But there is no reason for these functions
> to be at global scope.  It is more natural for them to be static factory
> functions in relocInfo.
> 
> Testing:
> mach5 tier1

Kim Barrett 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 two additional commits since the last revision:

 - Merge branch 'master' into relocinfo-friends
 - make friend functions instead be static members

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11753/files
  - new: https://git.openjdk.org/jdk/pull/11753/files/90893679..86f991f3

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=11753&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=11753&range=00-01

  Stats: 4233 lines in 239 files changed: 2810 ins; 757 del; 666 mod
  Patch: https://git.openjdk.org/jdk/pull/11753.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11753/head:pull/11753

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


More information about the hotspot-compiler-dev mailing list