RFR: 8176472: Lazily create ModuleEntryTable

[Rachel Protacio]

Thanks, Harold (and George)!

has_reads_list() sounds reasonable to me. Here's that webrev: 
http://cr.openjdk.java.net/~rprotacio/8176472.01/

Thanks,

Rachel


On 4/14/2017 11:03 AM, harold seigel wrote:
> Hi Rachel,
>
> The changes look good!  They will definitely save memory.
>
> Would it be useful to rename ModuleEntry::has_reads() to something 
> like ModuleEntry::has_reads_list() ?
>
> Thanks, Harold
>
>
> On 4/14/2017 10:34 AM, Rachel Protacio wrote:
>> Hello!
>>
>> Please review this enhancement which moves the unnamed module out of 
>> the ModuleEntryTable and into the ClassLoaderData so that the MET can 
>> be lazily created only when other modules are present. The 
>> PackageTable size has also been updated to be more realistic. These 
>> two changes save the VM from losing memory space unnecessarily in the 
>> module data structures.
>>
>> Tested with JPRT and RBT tiers 2-5.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8176472
>> Open webrev: http://cr.openjdk.java.net/~rprotacio/8176472.00/
>>
>> Thank you,
>> Rachel
>