RFR (XS) 8143615 compactHashtable.hpp includes oop.inline.hpp file
stefan.karlsson at oracle.com
Thu Dec 3 09:09:09 UTC 2015
Thanks a lot for fixing this!
Would you mind swapping these lines around?:
Except for that nit, this looks good.
As David says, we should get rid of the allocation.inline.hpp include as
well, but I think we could defer that to a separate patch (if you want to).
On 2015-12-03 03:32, Ioi Lam wrote:
> Please review a very small fix:
> Bug: compactHashtable.hpp includes oop.inline.hpp file
> 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
> - Ioi
More information about the hotspot-dev