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

Thomas Stüfe thomas.stuefe at gmail.com
Thu Mar 8 09:22:40 UTC 2018


Hi Stefan,

thanks, this is a good cleanup.

Sometimes I wish there were a method to automatically strip code from
unnecessary includes.

Thanks, Thomas



On Wed, Mar 7, 2018 at 11: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
>


More information about the hotspot-dev mailing list