RFR: 8209186: Rename SimpleThresholdPolicy to TieredThresholdPolicy

Claes Redestad claes.redestad at oracle.com
Thu Aug 9 14:21:10 UTC 2018


Hi Per,

On 2018-08-09 15:43, Per Liden wrote:
> Hi Claes,
>
> On 08/09/2018 12:38 PM, Claes Redestad wrote:
>> Hi,
>>
>> Nils recently reminded me that we should rename the 
>> SimpleThresholdPolicy following the merge with 
>> AdvancedThresholdPolicy[0].
>>
>> Webrev: http://cr.openjdk.java.net/~redestad/8209186/open.00/
>
> The include order in compilationPolicy.cpp is now slightly incorrect:
>
>   ...
>   37 #include "runtime/frame.hpp"
>   38 #include "runtime/handles.inline.hpp"
>   39 #include "runtime/rframe.hpp"
>   40 #include "runtime/tieredThresholdPolicy.hpp" <------------
>   41 #include "runtime/stubRoutines.hpp"
>   42 #include "runtime/thread.hpp"
>   43 #include "runtime/timer.hpp"
>   44 #include "runtime/vframe.hpp"
>   ...

thanks for spotting this! I've moved the include to after thread.hpp and 
rebuilt without issue.

/Claes


More information about the hotspot-compiler-dev mailing list