RFR: 8347006: LoadRangeNode floats above array guard in arraycopy intrinsic [v2]

Tobias Hartmann thartmann at openjdk.org
Wed Jan 8 13:08:54 UTC 2025


On Wed, 8 Jan 2025 12:26:44 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> I wondered as well and no, we don't use this type anywhere else (the closest would be `TypeAryPtr::RANGE`). We only create it when meeting arrays of primitive and non-primitive element type. Do you think this should still go to `TypeAryPtr::*`?
>
> I would add it to `TypeAryPtr` (maybe as `TypeAryPtr::BOTTOM`) . The main benefit I see is that the new code would be more readable if it referred to `TypeAryPtr::BOTTOM` rather than the long type creation expression.

Sounds good, I'll update the patch accordingly.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22967#discussion_r1907151707


More information about the hotspot-compiler-dev mailing list