> VarHandle reflectively checks if a class has oops (and does it incorrectly by 
> not filtering static fields). This information is easily accessible from acmp 
> maps, and we should use that instead of spinning our custom logic.

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 five additional commits since 
the last revision:

 - Merge branch 'lworld' of https://github.com/openjdk/valhalla into 
fix/array-vh-acmp
 - Update terminology because everyone seems confused as hell
 - Refactors per comment
 - interface-typed fields were using atomic flat VHs ???
 - 8378273: [lworld] Use acmp map to detect oops for VarHandle

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2143/files
  - new: https://git.openjdk.org/valhalla/pull/2143/files/e36eff38..46b9d4c4

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2143&range=04
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2143&range=03-04

  Stats: 322 lines in 51 files changed: 125 ins; 111 del; 86 mod
  Patch: https://git.openjdk.org/valhalla/pull/2143.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2143/head:pull/2143

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

Reply via email to