Request for reviews (S): 7174510: 19 JCK compiler tests fail with C2 error: memNode.cpp:812 - ShouldNotReachHere

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Jun 8 17:14:38 PDT 2012


Thank you, Christian

Vladimir

Christian Thalinger wrote:
> On Jun 8, 2012, at 3:56 PM, Vladimir Kozlov wrote:
> 
>> http://cr.openjdk.java.net/~kvn/7174510/webrev
>>
>> Add missing check for EncodeP node in MemNode::Ideal_common_DU_postCCP() method.
> 
> +         // Use it's control edge for memory op because EncodeP may go away
> 
> Small typo "its".
> 
> Otherwise looks good.
> 
> -- Chris
> 
>> Tested with failing tests.
>>
>> Thanks,
>> Vladimir
> 


More information about the hotspot-compiler-dev mailing list