RFR (XS): 8035585: Restore UseOldInlining in jdk8u

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Feb 21 11:39:02 PST 2014


Kris,

The flag name is deceiving. We only used code with UseOldInlining=true.
The code which was removed was under !UseOldInlining (negated). We kept 
the code which was under UseOldInlining. So the behavior did not 
changed. And the flag is in deprecated list:

+ { "UseOldInlining", JDK_Version::jdk(9), JDK_Version::jdk(10) },

Thanks,
Vladimir

On 2/21/14 11:15 AM, Krystal Mok wrote:
> Hi Vladimir,
>
> Restoring a flag without backing code is a little bit weird. Is is
> possible to put it in the deprecated flags list instead?
>
> Thanks,
> Kris
>
>
> On Fri, Feb 21, 2014 at 10:41 AM, Vladimir Kozlov
> <vladimir.kozlov at oracle.com <mailto:vladimir.kozlov at oracle.com>> wrote:
>
>     https://bugs.openjdk.java.net/__browse/JDK-8035585
>     <https://bugs.openjdk.java.net/browse/JDK-8035585>
>     http://cr.openjdk.java.net/~__kvn/8035585/webrev/
>     <http://cr.openjdk.java.net/~kvn/8035585/webrev/>
>
>     I backported 8032490 changes which removed UseOldInlining flags. We
>     can't do that in update release.
>     Restore the flag but keep code changes. The removed code was never
>     used, tested and was rotten.
>
>     Thanks,
>     Vladimir
>
>


More information about the hotspot-compiler-dev mailing list