[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