RFR (XS) 8143615 compactHashtable.hpp includes oop.inline.hpp file

David Holmes david.holmes at oracle.com
Thu Dec 3 04:49:27 UTC 2015


Hi Ioi,

On 3/12/2015 12:32 PM, Ioi Lam wrote:
> Please review a very small fix:
>
> http://cr.openjdk.java.net/~iklam/8143615-compactHashtable-inline-hpp/
>
> Bug: compactHashtable.hpp includes oop.inline.hpp file
>
>      https://bugs.openjdk.java.net/browse/JDK-8143615
>
> Summary of fix:
>
>      + deleted the include of oop.inline.hpp from compactHashtable.hpp
>      + moved all inlined functions that require oop.inline.hpp into a
>        new file, compactHashtable.inline.hpp

Looks okay but in compactHashtable.hpp:

#include "memory/allocation.inline.hpp"

This also needs removing - right?

Thanks,
David

> Tests:
>
>      JPRT
>
> Thanks
> - Ioi


More information about the hotspot-dev mailing list