Integrated: Fix recent merge errors
Aleksey Shipilev
shade at openjdk.org
Tue Jun 3 07:03:19 UTC 2025
On Mon, 2 Jun 2025 20:12:18 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> I skipped through current premain-vs-mainline webrev, and noticed a few merge errors.
>
> `metaspaceShared.cpp` hunks are obvious duplications. I removed the lines that are actually different from mainline.
>
> ~`compilationPolicy.hpp` hunk is a leftover from AOT Profiling. I remember pointing this out to @veresov during mainline review, and we went into mainline without that extra `notify_all`. We should ditch it in premain too.~ Actually, I see test timeouts when that `notify_all` is removed. Filed [JDK-8358343](https://bugs.openjdk.org/browse/JDK-8358343) to figure it out. Yanked the hunk from this PR.
>
> Additional testing:
> - [x] Linux x86_64 server fastdebug, `runtime/cds`
This pull request has now been integrated.
Changeset: dd845008
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/leyden/commit/dd845008ac031c070cd2b282572eeeac4ef978f1
Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod
Fix recent merge errors
Reviewed-by: iveresov, kvn
-------------
PR: https://git.openjdk.org/leyden/pull/72
More information about the leyden-dev
mailing list