RFR (XS): 8065134: Need WhiteBox::allocateCodeBlob(long, int) method to be implemented

Vladimir Kozlov vladimir.kozlov at oracle.com
Thu Dec 4 17:01:59 UTC 2014


I think you need to add cast from long to int:

949 WB_ENTRY(jlong, WB_AllocateCodeBlob(JNIEnv* env, jobject o, jlong size, jint blob_type))
950     return (jlong) WhiteBox::allocate_code_blob((int)size, blob_type);

since:

WhiteBox::allocate_code_blob(int size, int blob_type) {

Otherwise some C++ compiler may complain.

Thanks,
Vladimir

On 12/4/14 8:34 AM, Tatiana Pivovarova wrote:
> Hi all,
>
> please review this small patch
>
> bugid: https://bugs.openjdk.java.net/browse/JDK-8065134
> webrev: http://cr.openjdk.java.net/~iignatyev/tpivovarova/8065134/webrev.00
>
> Problem:
> Need WhiteBox::allocateCodeBlob(long, int), currently only WhiteBox::allocateCodeBlob(int, int) exist.
>
> Solution:
> Just change function signature WhiteBox::allocateCodeBlob(int, int) -> WhiteBox::allocateCodeBlob(long, int)
>
> Test: jprt
>
> Thanks,
> Tatiana


More information about the hotspot-compiler-dev mailing list