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

Vladimir Kozlov vladimir.kozlov at oracle.com
Mon Dec 8 17:06:06 UTC 2014


The webrev is wrong, should be

http://cr.openjdk.java.net/~ppunegov/tpivovarova/8065134/webrev.03/

which looks good.

Thanks,
Vladimir

On 12/8/14 8:49 AM, Tatiana Pivovarova wrote:
> Hi all,
>
> please review this new patch
>
> bugid: https://bugs.openjdk.java.net/browse/JDK-8065134
> webrev: http://cr.openjdk.java.net/~ppunegov/tpivovarova/8065134/webrev.02/
>
> Problem:
> Need WhiteBox::allocateCodeBlob(long, int), currently only WhiteBox::allocateCodeBlob(int, int) exist.
>
> Solution:
> Add new function with check:
>
>   156   public        long    allocateCodeBlob(long size, int type) {
>   157       int intSize = (int) size;
>   158       if ((long) intSize != size || size < 0) {
>   159           throw new IllegalArgumentException(
>   160                 "size argument has illegal value " + size);
>   161       }
>   162       return allocateCodeBlob( intSize, type);
>   163   }
>
>
> Test: jprt
>
>
> Thanks,
> Tatiana


More information about the hotspot-compiler-dev mailing list