RFR: 8202711: Merge tiered compilation policies

Vladimir Kozlov vladimir.kozlov at oracle.com
Mon May 7 19:25:49 UTC 2018


Hi Cleas,

First, please explain why tiered compilation policies "should be merged 
into a single policy"?

After these changes there is no difference between 
CompilationPolicyChoice 2 and 3. Why you kept both?

Changes looks good but I wish all compilation policy files moved into 
hostpot/share/compiler directory. But it could be done separately 
otherwise current review will be difficult.

Good.

Thanks,
Vladimir

On 5/7/18 4:22 AM, Claes Redestad wrote:
> Hi,
> 
> the simple and advanced threshold policy for tiered compilation should 
> be merged into a single policy.
> 
> I suggest merging all redefined code in AdvancedThresholdPolicy into 
> SimpleThresholdPolicy:
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8202711
> Webrev: http://cr.openjdk.java.net/~redestad/8202711/open.00/
> 
> As a follow-up we should consider renaming this to TieredCompPolicy (to 
> be consistent with NonTieredCompPolicy) or TieredCompilationPolicy.
> 
> Thanks!
> 
> /Claes


More information about the hotspot-compiler-dev mailing list