RFR(S) 8251560 Remove excessive header file inclusion from systemDictionary.hpp and others
Ioi Lam
ioi.lam at oracle.com
Sat Aug 15 01:21:29 UTC 2020
Hi Coleen,
Thanks for the review.
Since the constructors are pretty simple, I moved them from
systemDictionary.cpp into classLoadInfo.hpp. I also removed the default
constructor which doesn't seem to be used.
http://cr.openjdk.java.net/~iklam/jdk16/8251560-remove-includes-in-system-dict-etc.v02-delta/
Thanks
- Ioi
On 8/14/20 11:38 AM, Coleen Phillimore wrote:
>
>
> On 8/14/20 2:35 PM, Coleen Phillimore wrote:
>>
>> I really like this change. It looks like you need a
>> classLoadInfo.hpp file with the constructors.
> Oops, I mean classLoadInfo.cpp file.
>>
>> Thanks,
>> Coleen
>>
>> On 8/14/20 12:54 PM, Ioi Lam wrote:
>>> https://bugs.openjdk.java.net/browse/JDK-8251560
>>> http://cr.openjdk.java.net/~iklam/jdk16/8251560-remove-includes-in-system-dict-etc.v01/
>>>
>>>
>>> Summary:
>>>
>>> I removed a few unnecessary #includes from systemDictionary.hpp,
>>> signature.hpp, altHashing.hpp and klass.inline.hpp
>>>
>>> Two rarely used classes are moved to a new header file
>>> classLoadInfo.hpp
>>>
>>> Some non-critical inline functions were moved to cpp files.
>>>
>>> Some cpp files needed to be updated (because they didn't explicitly
>>> declare dependencies on header files that were transitively included
>>> by other header files).
>>>
>>> Tested with mach5 tier1 and also local Linux builds (no-pch, minimal
>>> VM, etc).
>>> Thanks
>>> - Ioi
>>
>
More information about the hotspot-runtime-dev
mailing list