RFR: 8199275: Fix inclusions of allocation.inline.hpp
Stefan Karlsson
stefan.karlsson at oracle.com
Thu Mar 8 09:25:00 UTC 2018
Thanks Thomas.
StefanK
On 2018-03-08 10:22, Thomas Stüfe wrote:
> 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 <mailto: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/
> <http://cr.openjdk.java.net/%7Estefank/8199275/webrev.01/>
> https://bugs.openjdk.java.net/browse/JDK-8199275
> <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