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

Thomas Schatzl thomas.schatzl at oracle.com
Tue May 19 15:23:30 UTC 2015


Hi,

On Tue, 2015-05-19 at 10:11 +0200, Per Liden wrote:
> On 2015-05-19 02:11, Kim Barrett wrote:
> > 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
> >>
> [...]
> > 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.
> >
> 
> Good point. Here's an updated webrev, which includes all dependencies:
> 
> http://cr.openjdk.java.net/~pliden/8080584/webrev.1/

  looks good.

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list