[9] RFR (M): 8035828: Turn on @Stable support in VM

Christian Thalinger christian.thalinger at oracle.com
Fri Feb 28 08:22:29 PST 2014


Looks good.

On Feb 28, 2014, at 4:24 AM, Vladimir Ivanov <vladimir.x.ivanov at oracle.com> wrote:

> Chris, John, thanks for review.
> 
> Updated webrev:
> http://cr.openjdk.java.net/~vlivanov/8035828/webrev.01/
> 
> Best regards,
> Vladimir Ivanov
> 
> On 2/28/14 12:24 AM, John Rose wrote:
>> I agree; flags should be adjustable in product builds for bug or perf analysis, hence diagnostic.
>> 
>> Otherwise, thumbs up.
>> 
>> Nice tests!
>> 
>> — John
>> 
>> On Feb 27, 2014, at 10:06 AM, Christian Thalinger <christian.thalinger at oracle.com> wrote:
>> 
>>> Maybe both flags should be diagnostic?  At least FoldStableValues.  It’s nothing a regular customer would turn off on a daily basis but it might be helpful for diagnosing problems.
>>> 
>>> On Feb 26, 2014, at 6:25 AM, Vladimir Ivanov <vladimir.x.ivanov at oracle.com> wrote:
>>> 
>>>> http://cr.openjdk.java.net/~vlivanov/8035828/webrev.00/
>>>> https://bugs.openjdk.java.net/browse/JDK-8035828
>>>> 
>>>> @Stable was added in 8 (back in September, 2013) as an experimental feature. Future JSR292 performance improvements rely on it, so it should be turned on by default.
>>>> 
>>>> Also, as part of this change, I integrate unit tests on @Stable. They were part of initial implementation [1], but were broken when the annotation was moved to java.lang.invoke and made package-private. It blocked their integration. Now they are fixed.
>>>> 
>>>> Thanks!
>>>> 
>>>> Best regards,
>>>> Vladimir Ivanov
>>>> 
>>>> [1] http://cr.openjdk.java.net/~vlivanov/8001107/webrev.04/
>>> 
>> 



More information about the hotspot-compiler-dev mailing list