[lworld] Integrated: 8283777: [lworld] TestArrayCopyAsLoadsStores.java triggers assert after merging jdk-19+15

Tobias Hartmann thartmann at openjdk.java.net
Fri Jun 10 06:11:59 UTC 2022


On Thu, 9 Jun 2022 14:20:34 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> We hit a dominance assert because `maybe_cast_profiled_obj` replaces `original` in the map while there can still be a bailout afterwards. This code is not Valhalla specific but was moved up to before the uncommon trap long ago with https://github.com/openjdk/valhalla/commit/09f099cab74c483c4aee3e927f2bc3c41ad784c9. I don't think that's required anymore, so this change simply restores the mainline version.
> 
> Thanks,
> Tobias

This pull request has now been integrated.

Changeset: 02985564
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/02985564a67dc91a3f76d008b63b6dcdd786f6b8
Stats:     46 lines in 2 files changed: 20 ins; 24 del; 2 mod

8283777: [lworld] TestArrayCopyAsLoadsStores.java triggers assert after merging jdk-19+15

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

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



More information about the valhalla-dev mailing list