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

[Christian Thalinger]

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