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

[Tobias Hartmann]

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