RFR: 8199275: Fix inclusions of allocation.inline.hpp

Stefan Karlsson stefan.karlsson at oracle.com
Thu Mar 8 06:33:01 UTC 2018


Thanks Kim.

StefanK

On 2018-03-08 04:54, Kim Barrett wrote:
>> On Mar 7, 2018, at 5:33 PM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
>>
>> Hi all,
>>
>> Please review this small patch to fix some includes of allocation.inline.hpp.
>>
>> http://cr.openjdk.java.net/~stefank/8199275/webrev.01/
>> https://bugs.openjdk.java.net/browse/JDK-8199275
>>
>> The changes are quite simple:
>>
>> 1) SymbolHashMap::~SymbolHashMap and CDSOffsets::CDSOffsets were moved to .cpp files, since they used functions from allocation.inline.hpp.
>>
>> 2) includes of allocation.inline.hpp were added to .cpp and .inline.hpp files that used functions from allocation.inline.hpp
>>
>> The patch contains a few number added includes need after this restructuring.
>>
>> Thanks,
>> StefanK
> Looks good.
>



More information about the hotspot-dev mailing list