Request for review (S): 7042122: JSR 292: adjust various inline thresholds for JSR 292 API methods and method handle adapters
tom.rodriguez at oracle.com
Mon May 9 09:42:16 PDT 2011
That sounds good to me.
On May 9, 2011, at 7:41 AM, Christian Thalinger wrote:
> On May 7, 2011, at 12:29 AM, John Rose wrote:
>> On May 6, 2011, at 2:42 PM, Vladimir Kozlov wrote:
>>> OK, then it makes sense to have MDO. An other thing. You actually don't need real MDO. Do you? As I understand you need MDO only during compilation then you should use CURRENT_ENV->get_empty_methodData() in ciMethodHandle::get_adapter()
>>> to set call counters and assign it to created ciMethod._method_data.
>> It's probably safer to have the real MDO. The adapter is a real method, and it can (I think) reappear in the course of deoptimization and maybe even recompilation.
>> (This raises the question of whether we should be throwing away the adapters generated during compilation. It seems non-parsimonious to do so, but it is simplest.)
> Can we agree on this as a point-fix? I guess we have to adjust it later anyway as we get more customer feedback (or we are doing more optimizations).
> -- Christian
More information about the hotspot-compiler-dev