Review request (7u) JDK-8038440: backport few C2 fixes

Poonam Bajaj poonam.bajaj at oracle.com
Wed Apr 2 01:21:36 UTC 2014


Hi Vladimir,

Sorry, mistake. I have corrected it. Please check the following webrev:
http://cr.openjdk.java.net/~poonam/8038440/webrev.01/

Thanks,
Poonam

On 3/31/2014 11:37 PM, Vladimir Kozlov wrote:
> Poonam,
>
> Why you did not remove original UseOptoBiasInlining lines in 
> arguments.cpp?
>
> Thanks,
> Vladimir
>
> On 3/30/14 10:36 PM, Poonam Bajaj wrote:
>> Bug: JDK-8038440 <https://bugs.openjdk.java.net/browse/JDK-8038440>: 
>> backport few C2 fixes
>> Webrev: http://cr.openjdk.java.net/~poonam/8038440/webrev.00/
>>
>> These changes backport few C2 fixes to 7u. Changes backported:
>> 1. Move disabling UseOptoBiasInlining after the last change to 
>> UseBiasedLocking in src/share/vm/runtime/arguments.cp.
>> 2. In OptoRuntime::new_named_counter() in 
>> src/share/vm/opto/runtime.cpp file, cmpxchg may fail first time when 
>> new named
>> counter is added. It triggers assert in set_next() because _next is 
>> set already. Fix it by resetting _next for each attempt.
>>
>> Testing:
>> JPRT
>>
>> Thanks,
>> Poonam
>>


More information about the hotspot-compiler-dev mailing list