CRR (S): 8058495: G1: normalize names for isHumongous() and friends.

Thomas Schatzl thomas.schatzl at oracle.com
Tue Sep 16 08:47:01 UTC 2014


Hi,

On Mon, 2014-09-15 at 13:52 -0400, Tony Printezis wrote:
> Hi all,
> 
> This is a follow-up to JDK-8057768, proposed by Thomas. It normalizes 
> some of the humongous-related method names (and enums). The changes are:
> 
> HeapRegion::isHumongous() -> is_humongous()
> HeapRegion::startsHumongous() -> is_starts_humongous()
> HeapRegion::continuesHumongous() -> is_continues_humongous()
> 
> HeapRegion::set_startsHumongous() -> set_starts_humongous()
> HeapRegion::set_continuesHumongous() -> set_continues_humongous()
> 
> HeapRegionType::HumMask -> HumongousMask
> HeapRegionType::HumStartsTag -> StartsHumongousTag
> HeapRegionType::HumContTag -> ContinuesHumongousTag
> 
> G1CollectedHeap::isHumongous() -> is_humongous()
> 
> Webrev is here:
> 
> http://cr.openjdk.java.net/~tonyp/8058495/webrev.0/
> 
> Could I get a couple of reviews please (and a sponsor!!!)? :-)

 looks good.

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list