[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:


More information about the valhalla-dev mailing list