[9] RFR(S): 8042431: compiler/7200264/TestIntVect.java fails with: Test Failed: AddVI 0 < 4

Niclas Adlertz niclas.adlertz at oracle.com
Tue May 6 13:21:01 UTC 2014


Just a suggestion, but personally I would find this easier to read:

// case 1
if (no_compiler) {
   return 0;
}

// case 2 or 3
if (!TieredCompilation || TieredStopAtLevel < CompLevel_full_optimization) {
   return 1;
}

// case 4
return 2;

Kind Regards,
Niclas Adlertz

On 05/06/2014 03:04 PM, Niclas Adlertz wrote:
> Hi Albert,
>
> Inside src/share/vm/runtime/arguments.cpp:
> +   // We should not come here. However, we also do not fail the VM,
> since this is a not-critical error.
> +   assert (false, "Missed configuration to set the minimum number of
> compiler threads.");
> +   // Return 2, since this works with all configurations on all platforms.
> +   return 2;
>
> That is dead code, and we can remove it.
> If none of the cases 1, 2 or 3 are true, we will always enter case 4.
>
> Kind Regards,
> Niclas Adlertz
>
> On 05/06/2014 02:43 PM, Albert wrote:
>> Hi all,
>>
>> could I get reviews for this patch?
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8042431
>>
>> Problem:
>> The minimum number of compiler threads is set incorrectly.
>>
>> Solution:
>> Define a new function 'get_min_number_of_compiler_threads()' that
>> determines the minimum number of compiler threads  for a particular
>> platform/configuration.
>>
>> Testing:
>> Failing test case; zero; -XX:-TieredCompilation -XX:CICompilerCount=1
>>
>> Webrev:
>> http://cr.openjdk.java.net/~anoll/8042431/webrev.00/
>>
>> Many thanks in advance,
>> Albert
>>
>>


More information about the hotspot-compiler-dev mailing list