Request for review (S): 7042122: JSR 292: adjust various inline thresholds for JSR 292 API methods and method handle adapters

[Tom Rodriguez]

That sounds good to me.

tom

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