RFR(xs): 8080584: isGCActiveMark.hpp should not include parallelScavengeHeap.hpp

Kim Barrett kim.barrett at oracle.com
Tue May 19 00:11:51 UTC 2015

On May 18, 2015, at 10:31 AM, Per Liden <per.liden at oracle.com> wrote:
> Hi,
> This is a follow up patch to the GC directory restructure, which removes the unnecessary dependency between gc/shared/isGCActiveMark.hpp and gc/parallel/parallelScavengeHeap.hpp
> Webrev: http://cr.openjdk.java.net/~pliden/8080584/webrev.0/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8080584
> cheers,
> /Per

  28 #include "memory/allocation.hpp"

This file needs several more includes to be clearly standalone. It
references at least the following:

- CollectedHeap class
- Universe class
- assert macro

I don't know if dealing with this is out of scope for this changeset,
but mention it since the changes go beyond just eliminating the 
conditional dependency on parallelScavengeHeap.hpp. 


More information about the hotspot-gc-dev mailing list