Request for reviews (M): 6924259: Remove String.count,	String.offset fields
    Coleen Phillimore 
    coleen.phillimore at oracle.com
       
    Mon May 14 08:45:00 PDT 2012
    
    
  
Does java_lang_String::compute_offsets() have to be in 
systemDictionary?   Can it not be in JavaClasses::compute_offsets() 
function?   Can you add a comment why, like java_lang_Class has.
This looks fine.  Didn't review the c2 code.
Thanks,
Coleen
On 5/10/2012 2:11 PM, Vladimir Kozlov wrote:
> http://cr.openjdk.java.net/~kvn/6924259/webrev
>
> 6924259: Remove String.count/String.offset
>
> Allows a version of the String class that doesn't have count,offset 
> fields.
> These VM changes works with both versions of String class by checking 
> the presence of fields.
>
> Initial changes were done by Brian and Tom years ago. I addapted it 
> for current sources and tested it.
>
> Tested with NSK, regression tests, CTW, refworkload, CDS (class sharing).
>
> Thanks,
> Vladimir
    
    
More information about the hotspot-dev
mailing list