RFR: 8176472: Lazily create ModuleEntryTable

[Rachel Protacio]

Thanks, George!


On 4/14/2017 2:00 PM, George Triantafillou wrote:
> Hi Rachel,
>
> Looks good.
>
> -George
>
> On 4/14/2017 1:48 PM, Rachel Protacio wrote:
>> 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
>>>
>>
>