RFR: 8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass

[Roman Kennke]

Am 20.07.2017 um 10:44 schrieb Erik Helin:
> On 07/17/2017 02:07 PM, Roman Kennke wrote:
>>>> Ok, added those and some more that I found. Not sure why we'd need
>>>> #include "gc/cms/concurrentMarkSweepGeneration.hpp" ? Left that out
>>>> for now.
>>>
>>> Because you are accessing CMSCollcetor in:
>>>
>>>  99   NOT_PRODUCT(
>>>  100     virtual size_t skip_header_HeapWords() { return
>>> CMSCollector::skip_header_HeapWords(); }
>>>  101   )
>>>
>>> and CMSCollector is declared in concurrentMarkSweepGeneration.hpp. An
>>> alternative would of course be to just declare skip_header_HeapWords()
>>> in cmsHeap.hpp and define skip_header_HeapWords in cmsHeap.cpp, then
>>> you only need to include concurrentMarkSweeoGeneration.hpp in
>>> cmsHeap.cpp.
>> Ah ok, I've missed that one. Added it now.
>
> Where did you add it? I don't see any include of
> "gc/cms/concurrentMarkSweepGeneration.hpp" in cmsHeap.hpp?

Hmm. I honestly don't know how that disappeared :-)

Differential:
http://cr.openjdk.java.net/~rkennke/8179387/webrev.09.diff/
<http://cr.openjdk.java.net/%7Erkennke/8179387/webrev.09.diff/>
Full:
http://cr.openjdk.java.net/~rkennke/8179387/webrev.09/
<http://cr.openjdk.java.net/%7Erkennke/8179387/webrev.09/>

I hope it's ok now.

Cheers, Roman