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

Vladimir Kozlov vladimir.kozlov at oracle.com
Thu May 30 14:51:28 PDT 2013


 >> +  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?

Could you split some checks into 2 lines in c1_Runtime1.cpp? They are 
too long.

Otherwise looks good, I think.

Vladimir

On 5/30/13 10:14 AM, Roland Westrelin wrote:
> New webrev:
> http://cr.openjdk.java.net/~roland/7199175/webrev.01/
>
> Roland.
>


More information about the hotspot-compiler-dev mailing list