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

[Vladimir Kozlov]

Okay, lets wait. It could be good exercise for Tobias.

Vladimir

On 4/15/14 12:54 PM, Igor Ignatyev wrote:
> Vladimir,
>
> I don't insist on having CCC, but my understating is what we have to
> file a CCC, since this flag can be available in product builds.
> Anyway Tobias already has filed a CCC request, so we can wait for the
> decision, just to be sure that we're OK.
>
> Igor
>
> On 04/15/2014 09:09 PM, Vladimir Kozlov wrote:
>> 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
>>>>