[lworld] RFR: 8348607: [lworld] Calls to Unsafe.getValue() / Unsafe.putValue() must be updated [v2]

Chen Liang liach at openjdk.org
Wed Jan 29 21:11:16 UTC 2025


> Core libraries counterpart of #1336.
> 
> This requires MemberName to fetch more information in initialization. A new int high flag field is added, in anticipation to a merger with the regular int flag field in the future. Currently, directly migrating the flags field to long is too impactful to the whole codebase.
> 
> Also 8348680 is addressed as well; it's some minor updates to make the null checks specific to references VH.
> 
> Testing: test/jdk/java/lang/invoke

Chen Liang 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 seven additional commits since the last revision:

 - Completely trust the value stored in fields (checked by verifier)
 - Merge branch 'lworld' of https://github.com/openjdk/valhalla into fix/vh-layout
 - Fix TestIntrinsics, safety remarks
 - Fix DMH too
 - Remove redundant checks for missing locations
 - New field in MemberName for now
 - Adapt layouts stage 1 (need to store in MemberName)

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/1340/files
  - new: https://git.openjdk.org/valhalla/pull/1340/files/1c08852b..a06cb9dc

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

  Stats: 82602 lines in 1449 files changed: 37468 ins; 38902 del; 6232 mod
  Patch: https://git.openjdk.org/valhalla/pull/1340.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1340/head:pull/1340

PR: https://git.openjdk.org/valhalla/pull/1340


More information about the valhalla-dev mailing list