RFR: 8305944: assert(is_aligned(ref, HeapWordSize)) failed: invariant
For narrowOop, the alignment should be sizeof(narrowOop) instead of HeapWordSize. ------------- Commit messages: - 8305944: assert(is_aligned(ref, HeapWordSize)) failed: invariant Changes: https://git.openjdk.org/jdk/pull/13471/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=13471&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8305944 Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod Patch: https://git.openjdk.org/jdk/pull/13471.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/13471/head:pull/13471 PR: https://git.openjdk.org/jdk/pull/13471
On Fri, 14 Apr 2023 03:48:02 GMT, SUN Guoyun <duke@openjdk.org> wrote:
For narrowOop, the alignment should be sizeof(narrowOop) instead of HeapWordSize.
Looks good. ------------- Marked as reviewed by stefank (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/13471#pullrequestreview-1384618282
On Fri, 14 Apr 2023 03:48:02 GMT, SUN Guoyun <duke@openjdk.org> wrote:
For narrowOop, the alignment should be sizeof(narrowOop) instead of HeapWordSize.
Marked as reviewed by tschatzl (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/13471#pullrequestreview-1384873742
On Fri, 14 Apr 2023 03:48:02 GMT, SUN Guoyun <duke@openjdk.org> wrote:
For narrowOop, the alignment should be sizeof(narrowOop) instead of HeapWordSize.
This pull request has now been integrated. Changeset: 3651b922 Author: sunguoyun <sunguoyun@loongson.cn> Committer: Stefan Karlsson <stefank@openjdk.org> URL: https://git.openjdk.org/jdk/commit/3651b92229afc78b274ff11c56e2da108ed25bec Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod 8305944: assert(is_aligned(ref, HeapWordSize)) failed: invariant Reviewed-by: stefank, tschatzl ------------- PR: https://git.openjdk.org/jdk/pull/13471
participants (3)
-
Stefan Karlsson
-
SUN Guoyun
-
Thomas Schatzl