[LWorld] Value type array support for C2 (and lots of bug fixes)

Tobias Hartmann tobias.hartmann at oracle.com
Thu May 17 14:24:52 UTC 2018


On 17.05.2018 16:07, Roland Westrelin wrote:
> I was thinking:
> 
> } else if (phi->bottom_type()->isa_valuetype() && !value->is_ValueType()) {
>   assert(value->bottom_type()->remove_speculative() == TypePtr::NULL_PTR, "Anything other than null?");
>   inc_sp(1);
>   uncommon_trap(Deoptimization::Reason_null_check, Deoptimization::Action_none);

You are right. I was assuming that 'value' can also be a non-null value type oop but that's not the
case. Here's the new incremental webrev:
http://cr.openjdk.java.net/~thartmann/valhalla/lworld_arrays/webrev.02

Thanks,
Tobias



More information about the valhalla-dev mailing list