RFR (S): 8074546: Rename and clean up the ParGCAllocBuffer class

Kim Barrett kim.barrett at oracle.com
Tue Apr 7 16:41:26 UTC 2015


On Apr 7, 2015, at 12:16 PM, Joseph Provino <joseph.provino at oracle.com> wrote:
> 
> I made the changes recommended by Kim.
> 
> It passes jprt.
> 
> As Kim mentioned, removing plab.h from g1CollectedHeap.hpp could potentially show
> places where plab.h should be included but isn't.
> 
> Is there something else I should do to verify there isn't a problem?

The problem I was suggesting might show up is that some file uses stuff in plab.hpp but only
includes g1CollectedHeap.hpp.  Removing the plab.hpp include from g1CollectedHeap.hpp
might result in build failures due to missing includes of plab.hpp.

If it builds (on all platforms), then it should be fine.



More information about the hotspot-gc-dev mailing list