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