Request for review (S): 7018355: JSR 292: VM crash in DefNewGeneration::copy_to_survivor_space

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Apr 15 08:42:06 PDT 2011


Looks fine.

Vladimir

Christian Thalinger wrote:
> John suggested:
> 
> "Consider changing return-by-reference of klassOop to rbr of a handle. Avoid the scoped oop temp."
> 
> So I changed MethodHandles::decode_* function to take a KlassHandle and return a methodHandle instead of oops which makes the uses of these methods simpler.
> 
> webrev udpated.
> 
> -- Christian
> 
> On Apr 13, 2011, at 11:11 PM, Christian Thalinger wrote:
>> On Apr 13, 2011, at 9:01 PM, Vladimir Kozlov wrote:
>>> Looks good.
>> Thanks, Vladimir.  -- Christian
>>
>>> Vladimir
>>>
>>> Christian Thalinger wrote:
>>>> [Resending.]
>>>> http://cr.openjdk.java.net/~twisti/7018355
>>>> 7018355: JSR 292: VM crash in DefNewGeneration::copy_to_survivor_space
>>>> Reviewed-by:
>>>> There are a couple of unhandled oop usages over potential safepoints
>>>> in methodHandles.cpp.  Three of them could be found with
>>>> +CheckUnhandledOops, the other ones were either found via
>>>> visual-inspection or just were changed for the sake of safety.
>>>> src/share/vm/memory/genOopClosures.hpp
>>>> src/share/vm/prims/methodHandles.cpp
> 
> 


More information about the hotspot-compiler-dev mailing list