[9] RFR (XXS): 8138861: C2 complains about unreasonably large method running Octane zlib in Nashorn

[Vladimir Kozlov]

Good.

Thanks,
Vladimir K

On 3/30/16 7:53 AM, Vladimir Ivanov wrote:
> https://bugs.openjdk.java.net/browse/JDK-8138861
>
> There's an assert on the maximum size of a nmethod produced by C2.
>
> I propose to simply remove it [1]. Maximum IR size limit (MaxNodeLimit) already establishes a limit on nmethod size
> (though indirectly).
>
> Testing: manual (zlib w/ fastdebug binaries).
>
> Best regards,
> Vladimir Ivanov
>
> [1]
> diff --git a/src/share/vm/opto/output.cpp b/src/share/vm/opto/output.cpp
> --- a/src/share/vm/opto/output.cpp
> +++ b/src/share/vm/opto/output.cpp
> @@ -1483,8 +1483,6 @@
>     // Compute the size of the first block
>     _first_block_size = blk_labels[1].loc_pos() - blk_labels[0].loc_pos();
>
> -  assert(cb->insts_size() < 500000, "method is unreasonably large");
> -
>   #ifdef ASSERT
>     for (uint i = 0; i < nblocks; i++) { // For all blocks
>       if (jmp_target[i] != 0) {