RFR: 8200105: Remove cyclic dependency between oop.inline.hpp and collectedHeap.inline.hpp

[coleen.phillimore at oracle.com]

http://cr.openjdk.java.net/~stefank/8200105/webrev.01/src/hotspot/share/oops/oop.inline.hpp.udiff.html
This is cool how every include line is 2 characters less than the previous.

On 3/22/18 12:10 PM, Stefan Karlsson wrote:
> Hi all,
>
> Please review this patch to remove a cyclic dependency between 
> oop.inline.hpp and collectedHeap.inline.hpp:
>
> http://cr.openjdk.java.net/~stefank/8200105/webrev.01/
> https://bugs.openjdk.java.net/browse/JDK-8200105
>
> The ground work for this cleanup had already been done with:
> 8199728: Remove oopDesc::is_scavengable
>
> What's done in this trivial patch is:
> 1) Move some debug functions out from the collectedHeap.inline.hpp and 
> into collectedHeap.hpp.

You mean into collectedHeap.cpp right?
>
> 2) Removed collectedHeap.inline.hpp and fixed missing includes to make 
> it compile again.

Looks good.   Less transitive includes.   I assume you ran open-only 
with --disable-precompiled-headers as well as the oracle platforms in mach5?

Zero build is probably unnecessary here (it builds now).

I concur that this is trivial.

Thanks,
Coleen
>
> Thanks,
> StefanK