-XX:+UseOldInlining

[Mikael Gerdin]

On Friday 10 January 2014 13.03.40 Mikael Gerdin wrote:
> On Friday 10 January 2014 12.23.32 Aleksey Shipilev wrote:
> > Hi,
> > 
> > Does anyone remember the story behind -XX:+UseOldInlining? I have
> > googled, wandered through HotSpot code, searched through JIRA, grepped
> > through Mercurial to no avail. The history on this predates OpenJDK.
> 
> If, in the future, you want to dig around in the prehistoric source
> management system, check out OpenGrok:
> http://opengrok.ie.oracle.com:8080/opengrok/search?q=UseOldInlining&project=
> jdk7b23_sccs&defs=&refs=&path=&hist=

Sorry, that's an internal url :(

> 
> /Mikael
> 
> > In seems we have it unconditionally turned on. It makes the inlining
> > policy a little less understandable to have additional branch here and
> > there. Should we purge this option from the VM codebase? If so, I can
> > 
> > prepare the proper revision of this scratch change:
> >   http://cr.openjdk.java.net/~shade/scratch/purge-old-inlining.patch
> > 
> > Thanks,
> > -Aleksey.