From: <dfr...@us...> - 2007-07-26 01:20:51
|
Revision: 13172 http://jikesrvm.svn.sourceforge.net/jikesrvm/?rev=13172&view=rev Author: dframpton-oss Date: 2007-07-25 18:20:49 -0700 (Wed, 25 Jul 2007) Log Message: ----------- Make MMTk Option classes final to improve codegen. Removes yieldPointForOSR in CollectorContext.collectionPhase. Modified Paths: -------------- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/BoundedNursery.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleFilterThreshold.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleMetaDataLimit.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleTriggerThreshold.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/DebugAddress.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EagerCompleteSweep.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EagerMmapSpaces.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EchoOptions.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FixedNursery.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FragmentationStats.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FullHeapSystemGC.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCTimeCap.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyPort.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyTileSize.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyWait.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GenCycleDetection.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/IgnoreSystemGC.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/MarkSweepMarkBits.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/MetaDataLimit.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NoFinalizer.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NoReferenceTypes.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NurserySize.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/PrintPhaseStats.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/ProtectOnRelease.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/SanityCheck.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/StressFactor.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/TraceRate.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VariableSizeHeap.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/Verbose.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VerboseFragmentationStats.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VerboseTiming.java rvmroot/trunk/MMTk/src/org/mmtk/utility/options/XmlStats.java Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/BoundedNursery.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/BoundedNursery.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/BoundedNursery.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -18,7 +18,7 @@ * Provide an upper bound on nursery size. This option is not intended to * be created directly, but via NurserySize. */ -public class BoundedNursery extends PagesOption { +public final class BoundedNursery extends PagesOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleFilterThreshold.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleFilterThreshold.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleFilterThreshold.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Trigger cycle buffer filtering if the space available falls below this threshold. */ -public class CycleFilterThreshold extends PagesOption { +public final class CycleFilterThreshold extends PagesOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleMetaDataLimit.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleMetaDataLimit.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleMetaDataLimit.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Trigger cycle detection if the meta data volume grows to this limit. */ -public class CycleMetaDataLimit extends PagesOption { +public final class CycleMetaDataLimit extends PagesOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleTriggerThreshold.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleTriggerThreshold.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/CycleTriggerThreshold.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Trigger cycle detection if the space available falls below this threshold. */ -public class CycleTriggerThreshold extends PagesOption { +public final class CycleTriggerThreshold extends PagesOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/DebugAddress.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/DebugAddress.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/DebugAddress.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -17,7 +17,7 @@ /** * Allow an address to be specified on the command line for use in debugging. */ -public class DebugAddress extends AddressOption { +public final class DebugAddress extends AddressOption { /** * Create the option */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EagerCompleteSweep.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EagerCompleteSweep.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EagerCompleteSweep.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should we eagerly finish sweeping at the start of a collection */ -public class EagerCompleteSweep extends BooleanOption { +public final class EagerCompleteSweep extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EagerMmapSpaces.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EagerMmapSpaces.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EagerMmapSpaces.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should spaces be eagerly demand zero mmapped? */ -public class EagerMmapSpaces extends BooleanOption { +public final class EagerMmapSpaces extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EchoOptions.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EchoOptions.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/EchoOptions.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Echo when options are set? */ -public class EchoOptions extends BooleanOption { +public final class EchoOptions extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FixedNursery.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FixedNursery.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FixedNursery.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -19,7 +19,7 @@ * * This option is not intended to be created directly, but via NurserySize. */ -public class FixedNursery extends PagesOption { +public final class FixedNursery extends PagesOption { // values BoundedNursery boundedNursery; Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FragmentationStats.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FragmentationStats.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FragmentationStats.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Option to print fragmentation information for the free list. */ -public class FragmentationStats extends BooleanOption { +public final class FragmentationStats extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FullHeapSystemGC.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FullHeapSystemGC.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/FullHeapSystemGC.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should a major GC be performed when a system GC is triggered? */ -public class FullHeapSystemGC extends BooleanOption { +public final class FullHeapSystemGC extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCTimeCap.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCTimeCap.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCTimeCap.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Try to limit reference counting collections to this time cap. */ -public class GCTimeCap extends MicrosecondsOption { +public final class GCTimeCap extends MicrosecondsOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyPort.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyPort.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyPort.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Port number for GCSpy server to connect with visualiser. */ -public class GCspyPort extends IntOption { +public final class GCspyPort extends IntOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyTileSize.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyTileSize.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyTileSize.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * GCspy Tile Size. */ -public class GCspyTileSize extends IntOption { +public final class GCspyTileSize extends IntOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyWait.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyWait.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GCspyWait.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should the VM wait for the visualiser to connect? */ -public class GCspyWait extends BooleanOption { +public final class GCspyWait extends BooleanOption { /** * Create the option */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GenCycleDetection.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GenCycleDetection.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/GenCycleDetection.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should we use a generational approach to cycle detection? */ -public class GenCycleDetection extends BooleanOption { +public final class GenCycleDetection extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/IgnoreSystemGC.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/IgnoreSystemGC.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/IgnoreSystemGC.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should we ignore calls to java.lang.System.gc? */ -public class IgnoreSystemGC extends BooleanOption { +public final class IgnoreSystemGC extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/MarkSweepMarkBits.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/MarkSweepMarkBits.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/MarkSweepMarkBits.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -17,7 +17,7 @@ /** * Number of bits to use for the header cycle of mark sweep spaces. */ -public class MarkSweepMarkBits extends IntOption { +public final class MarkSweepMarkBits extends IntOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/MetaDataLimit.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/MetaDataLimit.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/MetaDataLimit.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Provide a bound on how much metadata is allowed before a GC is triggered. */ -public class MetaDataLimit extends PagesOption { +public final class MetaDataLimit extends PagesOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NoFinalizer.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NoFinalizer.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NoFinalizer.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should finalization be disabled? */ -public class NoFinalizer extends BooleanOption { +public final class NoFinalizer extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NoReferenceTypes.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NoReferenceTypes.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NoReferenceTypes.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should reference type processing be disabled? */ -public class NoReferenceTypes extends BooleanOption { +public final class NoReferenceTypes extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NurserySize.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NurserySize.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/NurserySize.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -18,7 +18,7 @@ * A composite option that provides a min/max interface to MMTk, * and a fixed/bounded option interface to the VM/user. */ -public class NurserySize { +public final class NurserySize { // values private FixedNursery fixedNursery; private BoundedNursery boundedNursery; Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/PrintPhaseStats.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/PrintPhaseStats.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/PrintPhaseStats.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -16,7 +16,7 @@ * When printing statistics, should statistics for each * gc-mutator phase be printed? */ -public class PrintPhaseStats extends BooleanOption { +public final class PrintPhaseStats extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/ProtectOnRelease.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/ProtectOnRelease.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/ProtectOnRelease.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should memory be protected on release? */ -public class ProtectOnRelease extends BooleanOption { +public final class ProtectOnRelease extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/SanityCheck.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/SanityCheck.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/SanityCheck.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -16,7 +16,7 @@ /** * Should a major GC be performed when a system GC is triggered? */ -public class SanityCheck extends BooleanOption { +public final class SanityCheck extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/StressFactor.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/StressFactor.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/StressFactor.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -18,7 +18,7 @@ /** * Force frequent collections after amounts of allocation. */ -public class StressFactor extends PagesOption { +public final class StressFactor extends PagesOption { /** * Create the option, defaulting to the maximum possible value. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/TraceRate.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/TraceRate.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/TraceRate.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -17,7 +17,7 @@ /** * The granularity of the trace being produced. */ -public class TraceRate extends IntOption +public final class TraceRate extends IntOption implements org.mmtk.utility.Constants { /** * Create the option. Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VariableSizeHeap.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VariableSizeHeap.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VariableSizeHeap.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should we shrink/grow the heap to adjust to application working set? */ -public class VariableSizeHeap extends BooleanOption { +public final class VariableSizeHeap extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/Verbose.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/Verbose.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/Verbose.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * GC verbosity level. */ -public class Verbose extends IntOption { +public final class Verbose extends IntOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VerboseFragmentationStats.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VerboseFragmentationStats.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VerboseFragmentationStats.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should we print verbose fragmentation statistics for the free list allocator? */ -public class VerboseFragmentationStats extends BooleanOption { +public final class VerboseFragmentationStats extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VerboseTiming.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VerboseTiming.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/VerboseTiming.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -15,7 +15,7 @@ /** * Should we display detailed breakdown of where GC time is spent? */ -public class VerboseTiming extends BooleanOption { +public final class VerboseTiming extends BooleanOption { /** * Create the option. */ Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/options/XmlStats.java =================================================================== --- rvmroot/trunk/MMTk/src/org/mmtk/utility/options/XmlStats.java 2007-07-25 18:01:16 UTC (rev 13171) +++ rvmroot/trunk/MMTk/src/org/mmtk/utility/options/XmlStats.java 2007-07-26 01:20:49 UTC (rev 13172) @@ -16,7 +16,7 @@ * Display statistics and options in XML rather than himan-readable * format. */ -public class XmlStats extends BooleanOption { +public final class XmlStats extends BooleanOption { /** * Create the option. This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |