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

Kim Barrett kim.barrett at oracle.com
Thu Mar 8 03:54:33 UTC 2018


> 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