Request for review (S): 7042122: JSR 292: adjust various inline thresholds for JSR 292 API methods and method handle adapters
vladimir.kozlov at oracle.com
Fri May 6 08:48:21 PDT 2011
I don't see changing inline sizes for 292 methods. Do you not need it anymore? Also question, how many call sites mh
adapter has? I thought adapter has only one call site. Then why generate MDO only to propagate counter for one call site?
On 5/6/11 6:49 AM, Christian Thalinger wrote:
> On May 5, 2011, at 10:29 PM, Tom Rodriguez wrote:
>>>> The reason I came up with something like band-aid fix is because I didn't have enough time to do something more sophisticated and we need something that works in Java 7. JRuby wants to enable invokedynamic by default for the current development version as soon as we have heuristic that makes out-of-the-box performance good enough (reads: equally or better than non-indy).
>>>> Tom, do you have a better idea how we could handle that?
>>> Given where we are I thought your patch was a fine band aid. I was just a bit leery of adding a real exposed flag before we've really had a chance to make a policy. If we're fine with potentially abandoning that flag later then I'm fine with adding it.
>> I do have one concrete thought about this but it would require some investigation. The method handle chains themselves aren't profiled but presumably the base call site is. Maybe we should be looking back through method handle calls to find the frequency of the base call site and feeding that into the frequent call site logic.
> Yeah, I suppose that's how it should work. What about this?
> I get the same performance but without any tweaking of any inline thresholds.
> -- Christian
More information about the hotspot-compiler-dev