[lworld] RFR: 8377775: [lworld] compiler/ciReplay/TestInliningProtectionDomain.java still fails

Damon Fenacci dfenacci at openjdk.org
Mon Feb 16 15:10:41 UTC 2026


## Issue

The test `compiler/ciReplay/TestInliningProtectionDomain.java` still fails with `--enable-preview` even when using `java.lang.Record` as return class.

## Causes

`java.lang.Record` is part of the migrated value classes that are added to the class loader if _enable preview_ is on and consequently it gets resolved early, making `ProtectionDomainTestNoOtherCompilationPrivate` fail.

## Fix
To avoid further potential issues (with another lib class that could potentially be loaded early in the future) I replaced the `Record` return type with a new test-local `class SigType {}` in the three `bar()` methods. Additionally the private-case replay assertion had to be updated to expect `unloaded signature classes` instead of `disallowed by ciReplay` (`Record` was resolved early and inlining was explicitly denied by replay. `SigType` is simply not resolved). The purpose of the test (that replay compilation produces the same inlining outcome as the normal run) should still be met.

## Testing
Tier 1-3+

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

Commit messages:
 - Merge branch 'lworld' into JDK-8377775
 - JDK-8377775: [lworld] compiler/ciReplay/TestInliningProtectionDomain.java still fails

Changes: https://git.openjdk.org/valhalla/pull/2113/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=2113&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8377775
  Stats: 10 lines in 1 file changed: 3 ins; 0 del; 7 mod
  Patch: https://git.openjdk.org/valhalla/pull/2113.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2113/head:pull/2113

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


More information about the valhalla-dev mailing list