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