RFR(M): arraycopy/clone/copyOf intrinsics fixes for Lworld
Tobias Hartmann
tobias.hartmann at oracle.com
Mon Jun 4 07:04:19 UTC 2018
Hi Roland,
thanks for making that change!
In graphKit.cpp:3954 and macro.cpp:1443 shouldn't you use can_be_value_type() (you are not handling
the interface element type in macro.cpp)? Also, in macro.cpp you don't need to load elem_klass if
it's statically known.
Best regards,
Tobias
On 01.06.2018 17:13, Roland Westrelin wrote:
>
> Here is a new webrev. It implements your suggestion:
>
> http://cr.openjdk.java.net/~roland/valhalla/arraycopy-lworld/webrev.01/
>
> test63() is commented out for now because a bug in the type system
> causes an incorrect execution with non flattened arrays.
>
> Roland.
>
More information about the valhalla-dev
mailing list