[jdk8u-dev] RFR: 8233019: java.lang.Class.isPrimitive() (C1) returns wrong result if Klass* is aligned to 32bit [v4]

Dongbo He dongbohe at openjdk.org
Sat Jul 9 04:04:34 UTC 2022


> Hi,
> 
> I would like to backport this as follow up of [JDK-8150669](https://bugs.openjdk.java.net/browse/JDK-8150669)
> This resolves the corner case that leads to incorrect result for C1 intrinsic,
> 
> Original patch for 11u: https://hg.openjdk.java.net/jdk-updates/jdk11u-dev/rev/e1b6631cbd2f 
> Patch does not apply cleanly to 8u: arm and s390 ports are not there and we don’t have c1 compiler support in ppc port in 8u.
> 
> Performed full jtreg test both on x86_64-linux-gnu and aarch64-linux-gnu platforms. 
> (I made this PR on behalf of fyang)
> 
> Thanks,
> hedongbo

Dongbo He 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 master
 - Merge branch '8150669' into 8233019
 - Merge branch 'master' into 8150669
 - b67ca938f37f952e53f73d2e0b8ebcaf96139fda
 - Backport 890f94d7e6be731ac2ebae2f1ad3cc20ec836115

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

Changes:
  - all: https://git.openjdk.org/jdk8u-dev/pull/43/files
  - new: https://git.openjdk.org/jdk8u-dev/pull/43/files/5274a23f..69b5b2f8

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=43&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=43&range=02-03

  Stats: 507 lines in 22 files changed: 435 ins; 15 del; 57 mod
  Patch: https://git.openjdk.org/jdk8u-dev/pull/43.diff
  Fetch: git fetch https://git.openjdk.org/jdk8u-dev pull/43/head:pull/43

PR: https://git.openjdk.org/jdk8u-dev/pull/43


More information about the jdk8u-dev mailing list