Re: EnumData space optimization in j.l.Class (JEP-146)

2012-12-22 Thread Peter Levart
At end... On 12/18/2012 07:50 PM, Remi Forax wrote: On 12/18/2012 10:18 AM, Peter Levart wrote: On 12/17/2012 11:39 PM, Remi Forax wrote: On 12/17/2012 11:14 PM, Peter Levart wrote: On 12/17/2012 10:26 PM, Mandy Chung wrote: On 12/17/12 7:36 AM, Peter Levart wrote: Hi David and others, Her

Re: EnumData space optimization in j.l.Class (JEP-146)

2012-12-18 Thread Remi Forax
On 12/18/2012 10:18 AM, Peter Levart wrote: On 12/17/2012 11:39 PM, Remi Forax wrote: On 12/17/2012 11:14 PM, Peter Levart wrote: On 12/17/2012 10:26 PM, Mandy Chung wrote: On 12/17/12 7:36 AM, Peter Levart wrote: Hi David and others, Here's a patch that eliminates one of two fields in java

Re: EnumData space optimization in j.l.Class (JEP-146)

2012-12-18 Thread Peter Levart
On 12/17/2012 11:39 PM, Remi Forax wrote: On 12/17/2012 11:14 PM, Peter Levart wrote: On 12/17/2012 10:26 PM, Mandy Chung wrote: On 12/17/12 7:36 AM, Peter Levart wrote: Hi David and others, Here's a patch that eliminates one of two fields in java.lang.Class, related to caching enum constant

Re: EnumData space optimization in j.l.Class (JEP-146)

2012-12-17 Thread David Holmes
Hi Peter, BTW JEP-149 not 146! Sorry I didn't get a chance to respond last night before you continued on this path. I have to say no to this too. First I am just running out of time to get this finalized by M6 - particularly with the Xmas break looming. Second the trade-off here is far less

Re: EnumData space optimization in j.l.Class (JEP-146)

2012-12-17 Thread Remi Forax
On 12/17/2012 11:14 PM, Peter Levart wrote: On 12/17/2012 10:26 PM, Mandy Chung wrote: On 12/17/12 7:36 AM, Peter Levart wrote: Hi David and others, Here's a patch that eliminates one of two fields in java.lang.Class, related to caching enum constants: http://dl.dropbox.com/u/101777488/jdk8

Re: EnumData space optimization in j.l.Class (JEP-146)

2012-12-17 Thread Peter Levart
On 12/17/2012 10:26 PM, Mandy Chung wrote: On 12/17/12 7:36 AM, Peter Levart wrote: Hi David and others, Here's a patch that eliminates one of two fields in java.lang.Class, related to caching enum constants: http://dl.dropbox.com/u/101777488/jdk8-tl/JEP-149.enum/webrev.01/index.html It

Re: EnumData space optimization in j.l.Class (JEP-146)

2012-12-17 Thread Peter Levart
On 12/17/2012 10:26 PM, Mandy Chung wrote: On 12/17/12 7:36 AM, Peter Levart wrote: Hi David and others, Here's a patch that eliminates one of two fields in java.lang.Class, related to caching enum constants: http://dl.dropbox.com/u/101777488/jdk8-tl/JEP-149.enum/webrev.01/index.html It

Re: EnumData space optimization in j.l.Class (JEP-146)

2012-12-17 Thread Mandy Chung
On 12/17/12 7:36 AM, Peter Levart wrote: Hi David and others, Here's a patch that eliminates one of two fields in java.lang.Class, related to caching enum constants: http://dl.dropbox.com/u/101777488/jdk8-tl/JEP-149.enum/webrev.01/index.html It does it by moving one field to a subclass of

EnumData space optimization in j.l.Class (JEP-146)

2012-12-17 Thread Peter Levart
Hi David and others, Here's a patch that eliminates one of two fields in java.lang.Class, related to caching enum constants: http://dl.dropbox.com/u/101777488/jdk8-tl/JEP-149.enum/webrev.01/index.html It does it by moving one field to a subclass of HashMap, which is referenced by a remaining