[lworld] Integrated: 8280707: [lworld] More issues with C2's arraycopy intrinsic
Tobias Hartmann
thartmann at openjdk.java.net
Fri Jan 28 07:01:40 UTC 2022
On Thu, 27 Jan 2022 15:15:04 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
> The memory output of the `MemBarCPUOrder` node added to prevent writes into the source from floating below the arraycopy is not correctly wired. As a result, we hit an "dependence cycle found" assert in GCM. I also removed a comment in that code that was already removed from mainline.
>
> In `TypeAryPtr::add_field_offset_and_offset` we set the wrong offset if the field is not found in a flat array (for example, because the inline type has no fields). This leads to an assert in the matcher due to an unexpected address type change.
>
> Thanks,
> Tobias
This pull request has now been integrated.
Changeset: 22b271d0
Author: Tobias Hartmann <thartmann at openjdk.org>
URL: https://git.openjdk.java.net/valhalla/commit/22b271d0b3db95f914b9f8abb9eeb04169c1e05a
Stats: 93 lines in 3 files changed: 75 ins; 14 del; 4 mod
8280707: [lworld] More issues with C2's arraycopy intrinsic
-------------
PR: https://git.openjdk.java.net/valhalla/pull/620
More information about the valhalla-dev
mailing list