RFR(XS) 8145579: SimpleThresholdPolicy assumes non-trivial methods to be trivial

[dean.long at oracle.com]

Thanks Vladimir.

dl


On 12/4/17 2:11 PM, Vladimir Kozlov wrote:
> Good.
>
> Thanks,
> Vladimir
>
> On 12/4/17 9:44 AM, dean.long at oracle.com wrote:
>> https://bugs.openjdk.java.net/browse/JDK-8145579
>> http://cr.openjdk.java.net/~dlong/8145579/webrev/
>>
>> This change fixes certain non-trivial methods being marked as 
>> trivial.  A method should only be marked as trivial if C2 cannot 
>> generate better code.  The two cases fixed here are calling 
>> intrinsics, and Object.<init>, which has a hidden call to register 
>> finalizers.  When run with 
>> http://cr.openjdk.java.net/~shade/8145579/benchmarks.jar 
>> <http://cr.openjdk.java.net/%7Eshade/8145579/benchmarks.jar>, the 
>> test methods now end up compiled at level 4 instead of level 1.
>>
>> dl