[jdk25] RFR: 8361259: JDK25: Backout JDK-8258229
Dean Long
dlong at openjdk.org
Wed Jul 2 19:13:38 UTC 2025
On Wed, 2 Jul 2025 10:54:13 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:
> This is a backout of [JDK-8258229](https://bugs.openjdk.org/browse/JDK-8258229) for JDK25 only. The problematic code has already been removed by [JDK-8358821](https://bugs.openjdk.org/browse/JDK-8358821) in JDK26.
>
> The backout is clean for the C++ code, but the test backout includes the backout of the follow-up change [JDK-8356310](https://bugs.openjdk.org/browse/JDK-8356310).
>
> Rationale: Minimize risk for JDK25. We should use the better fix JDK-8358821 in the long term. However, that one should get some more stabilization time before backporting it. Also see JBS issue.
>
> Proposed long term solution: Backport JDK-8358821 to jdk25u and revert this change again after an appropriate time.
>
> Short term: The issue solved by JDK-8258229 is not critical. It should be ok to postpone the fix to jdk25u.
LGTM. I'm running Oracle testing now.
I'm not sure how to handle JDK-8357017 now in JBS. Close it as a duplicate of the backout? According to https://openjdk.org/guide, it sounds like it might have been more correct to use JDK-8357017 for the backout, and make it a subtask of JDK-8258229.
@TobiHartmann @JesperIRL , what do you think?
-------------
Marked as reviewed by dlong (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26091#pullrequestreview-2980343622
More information about the hotspot-compiler-dev
mailing list