[LWorld] Value type array support for C2 (and lots of bug fixes) (original) (raw)
Tobias Hartmann tobias.hartmann at oracle.com
Thu May 17 14:24:52 UTC 2018
- Previous message (by thread): [LWorld] Value type array support for C2 (and lots of bug fixes)
- Next message (by thread): [LWorld] Value type array support for C2 (and lots of bug fixes)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 17.05.2018 16:07, Roland Westrelin wrote:
I was thinking:
} else if (phi->bottomtype()->isavaluetype() && !value->isValueType()) { assert(value->bottomtype()->removespeculative() == TypePtr::NULLPTR, "Anything other than null?"); incsp(1); uncommontrap(Deoptimization::Reasonnullcheck, Deoptimization::Actionnone);
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
- Previous message (by thread): [LWorld] Value type array support for C2 (and lots of bug fixes)
- Next message (by thread): [LWorld] Value type array support for C2 (and lots of bug fixes)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]