On Mon, 16 Feb 2026 08:03:43 GMT, Damon Fenacci <[email protected]> wrote:
> ## 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+
This pull request has now been integrated.
Changeset: 4e8bfdb6
Author: Damon Fenacci <[email protected]>
URL:
https://git.openjdk.org/valhalla/commit/4e8bfdb622d60a24759781ad788db44b6bde5d98
Stats: 10 lines in 1 file changed: 3 ins; 0 del; 7 mod
8377775: [lworld] compiler/ciReplay/TestInliningProtectionDomain.java still
fails
Reviewed-by: thartmann
-------------
PR: https://git.openjdk.org/valhalla/pull/2113
