RFR (S): [MVT] Ensure JVMS spec rules for creation and loading of DVT are followed

David Simms david.simms at oracle.com
Wed Sep 13 09:32:44 UTC 2017


On 12/09/17 19:37, Frederic Parain wrote:
> Looks good to me.
>
> Minor point:
> src/share/vm/runtime/fieldType.cpp:
>
>   77 static const char dvt_postfix[] = "$Value";
>   78 static const int dvt_postfix_len = 6;
>
> Could we have an assert somewhere checking that dvt_postfix_len
> matches the dvt_postfix string? Just in case someone update the
> string without changing the len.
>
> Fred

Added debug assert...cheers, thanks for looking at this

/David Simms



More information about the valhalla-dev mailing list