RFR(M) 7199175: JSR 292: C1 needs patching when invokedynamic/invokehandle call site is not linked

[Vladimir Kozlov]

On 5/31/13 3:28 AM, Roland Westrelin wrote:
> Thanks for reviewing this, Vladimir.
>
>>>> +  develop(bool, C1PatchInvokeDynamic, true,       \
>>>>
>>>> Do we really want to turn if off?  During development I'm sure it was helpful but after that?
>>>
>>> That's something John asked for.
>>
>> I agree with Christian that this flag is weird. Why you would want to bailout even for testing?
>
> It's not a trivial change. So if tests that used to work start failing and this change is suspected, it will be as easy as running with -XX:-C1PatchInvokeDynamic to verify this change is the cause. I think it's a good idea to have this flag.

So you are saying that you are fixing performance issue only. Okay, may make it diagnostic then to be able use it with 
product VM?

>> Could you split some checks into 2 lines in c1_Runtime1.cpp? They are too long.
>
> Here is a new webrev in case you want to check I split the right checks.
>
> http://cr.openjdk.java.net/~roland/7199175/webrev.02/

This looks good.

Thanks,
Vladimir

>
> Roland.
>