RFR: 8316186: RISC-V: Remove PlatformCmpxchg<4> [v3]

Robbin Ehn rehn at openjdk.org
Thu Sep 28 11:07:10 UTC 2023


> Hi, please consider.
> 
> I don't know the history behind the Cmpxhg<4>, but it is not needed anymore.
> Using compiler as in 8 byte case is just fine.
> 
> Passes hotspot-tier (qemu, vf2 still running), some manual extra testing via gtest, looked at compiler: output, e.g. https://godbolt.org/z/a31Gdqn8q
> 
> If you know the history please share, and if you have a reason why we should keep it please speak up!

Robbin Ehn 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 three additional commits since the last revision:

 - Merge branch 'master' into 8316186-4cmpxchg
 - Remove redudant local
 - Removed

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15715/files
  - new: https://git.openjdk.org/jdk/pull/15715/files/6e5545aa..7a264fc0

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

  Stats: 73725 lines in 1541 files changed: 28023 ins; 12708 del; 32994 mod
  Patch: https://git.openjdk.org/jdk/pull/15715.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15715/head:pull/15715

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


More information about the hotspot-runtime-dev mailing list