[lworld] RFR: 8242402: multianewarray is missing checks on the bottom class
Harold Seigel
hseigel at openjdk.java.net
Mon Jul 6 20:14:45 UTC 2020
On Mon, 6 Jul 2020 20:09:59 GMT, Frederic Parain <fparain at openjdk.org> wrote:
>> src/hotspot/share/classfile/systemDictionary.cpp line 365:
>>
>>> 364: THROW_NULL(vmSymbols::java_lang_IncompatibleClassChangeError());
>>> 365: }
>>> 366: k = k->array_klass(ndims, CHECK_NULL);
>>
>> Is it worthwhile using THROW_MSG_NULL to provide a helpful message with the exception?
>
> Something like this?
>
> `THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), "L/Q mismatch on bottom type");`
>
> Fred
Yes. That looks good. Thanks.
-------------
PR: https://git.openjdk.java.net/valhalla/pull/105
More information about the valhalla-dev
mailing list