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