RFR: 8253970: Build error: address argument to atomic builtin must be a pointer to integer or pointer ('volatile narrowOop *' invalid) [v3]

Jie Fu jiefu at openjdk.java.net
Wed Oct 14 12:52:35 UTC 2020


> __sync_val_compare_and_swap shouldn't call with narrowOop* for clang after JDK-8247912.
> Before passing type T to __sync_val_compare_and_swap, the fix converts T to uint32_t* if sizeof(T) == 4.
> 
> Testing:
>  - Zero VM build on Linux and MacOS with clang
>  - Zero VM build on Linux with gcc

Jie Fu 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 eight additional commits since the last
revision:

 - Merge branch 'master' into JDK-8253970
 - Add FULL_MEM_BARRIER
 - Merge branch 'master' into JDK-8253970
 - Revert changes
 - Replace __sync_val_compare_and_swap whith __atomic_compare_exchange
 - Merge branch 'master' into JDK-8253970
 - Revert changes
 - 8253970 Build error: address argument to atomic builtin must be a pointer to integer or pointer ('volatile narrowOop *'
   invalid)

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/496/files
  - new: https://git.openjdk.java.net/jdk/pull/496/files/420ead76..9f0e1293

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=496&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=496&range=01-02

  Stats: 3878 lines in 87 files changed: 1268 ins; 1971 del; 639 mod
  Patch: https://git.openjdk.java.net/jdk/pull/496.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/496/head:pull/496

PR: https://git.openjdk.java.net/jdk/pull/496


More information about the hotspot-runtime-dev mailing list