Request for reviews (S): 7110832: ctw/.../org_apache_avalon_composition_util_StringHelper crashes the VM

[Igor Veresov]

Looks good! 

But I don't think you need this statement:
265 friend class ShortBranchVerifier;



igor


On Thursday, December 22, 2011 at 4:12 PM, Vladimir Kozlov wrote:

> http://cr.openjdk.java.net/~kvn/7110832/webrev
> 
> 7110832: ctw/.../org_apache_avalon_composition_util_StringHelper crashes the VM
> 
> Distance is too large for one short branch in string_indexofC8(). Note, it does 
> not fail always because different REX prefixes are generated for different input 
> registers.
> 
> I added verification code in debug VM by reducing allowed distance (by 32 bytes) 
> for short branches. It can be used in a large hand written assembler code.
> 
> Tested with full CTW.
> 
> Thanks,
> Vladimir