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

Roland Westrelin roland.westrelin at oracle.com
Fri May 31 03:28:05 PDT 2013


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.

> 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/

Roland.


More information about the hotspot-compiler-dev mailing list