[9] RFR(M): JDK-8036956: remove EnableInvokeDynamic flag

[Vladimir Kozlov]

Igor,

we don't need CCC because the flag is diagnostic and not product.

Thanks,
Vladimir

On 4/15/14 3:23 AM, Igor Ignatyev wrote:
> Hi Tobias,
>
> Do you have CCC for this?
>
> Igor
>
> On 04/15/2014 02:12 PM, Tobias Hartmann wrote:
>> Vladimir, Christian, thanks for the reviews!
>>
>>
>> On 04/14/2014 10:55 PM, Christian Thalinger wrote:
>>> On Apr 14, 2014, at 7:32 AM, Vladimir
>>> Kozlov<vladimir.kozlov at oracle.com>  wrote:
>>>
>>>> Hi Tobias,
>>>>
>>>> I think, in javaClasses.cpp computing _form_offset should not be
>>>> optional.
>>> Yes.
>> True, missed that one. Please find the updated webrev at:
>>
>> http://cr.openjdk.java.net/~anoll/8036956/webrev.01/
>> <http://cr.openjdk.java.net/%7Eanoll/8036956/webrev.01/>
>>
>> Thanks,
>> Tobias
>>
>>>> Otherwise it looks good.
>>> I agree.  Looks good.
>>>
>>>> Thanks,
>>>> Vladimir
>>>>
>>>> On 4/13/14 11:56 PM, Tobias Hartmann wrote:
>>>>> Hi,
>>>>>
>>>>> please review the following patch for bug 8036956.
>>>>>
>>>>> *Problem:*
>>>>> The VM option EnableInvokeDynamic is not longer used in JDK 9. It
>>>>> is enabled by default and it is not possible to run
>>>>> HotSpot in JDK9 without it (see bug description).
>>>>> Bug:https://bugs.openjdk.java.net/browse/JDK-8036956
>>>>> *
>>>>> **Solution:*
>>>>> The EnableInvokeDynamic flag and all support code is removed.
>>>>> Webrev:http://cr.openjdk.java.net/~anoll/8036956/webrev.00/
>>>>> <http://cr.openjdk.java.net/%7Eanoll/8036956/webrev.00/>
>>>>>
>>>>> *Tests:*
>>>>> JPRT
>>>>>
>>>>> Thanks,
>>>>> Tobias
>>