RFR(XS) : 8021120 : TieredCompilation can be enabled even if TIERED is undefined

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Jul 24 07:31:49 PDT 2013


On 7/24/13 4:29 AM, David Holmes wrote:
> On 24/07/2013 9:12 AM, Vladimir Kozlov wrote:
>> David,
>>
>> We can't use UNSUPPORTED_OPTION because of the message it produces: "is
>> disabled in this release." We need: "is not supported in this VM."
>
> Do the exact words really matter? If it is disabled it is not supported.
>
> But the patch issues no warning it simply ignores what the user has asked for. I prefer it when people get told they are
> asking for something they can't have.

Last version Igor sent has warning:

http://cr.openjdk.java.net/~iignatyev/8021120/webrev.01/

Vladimir

>
> David
>
>> Thanks,
>> Vladimir
>>
>> On 7/23/13 4:04 PM, David Holmes wrote:
>>> Hi Igor,
>>>
>>> On 23/07/2013 10:38 PM, Igor Ignatyev wrote:
>>>> Hi all,
>>>>
>>>> Please review patch.
>>>>
>>>> Problem:
>>>> If 'COMPILER2' is defined, it will be possible to enable
>>>> 'TieredCompilation' even if 'TIERED' is undefined. But such VM will
>>>> raise error at 'compilationPolicy_init'
>>>>
>>>> Fix:
>>>> Forcibly set 'TieredCompilation' in false, if 'TIERED' is undefined.
>>>
>>> If you use UNSUPPORTED_OPTION it will issue a warning and force it to
>>> false.
>>>
>>> David
>>> -----
>>>
>>>> webrev: http://cr.openjdk.java.net/~iignatyev/8021120/webrev.00/
>>>> jbs: https://jbs.oracle.com/bugs/browse/JDK-8021120
>>>> testing: jprt, 'java -version' on different VM (including embedded
>>>> client, server, minimal)


More information about the hotspot-compiler-dev mailing list