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

[dean.long at oracle.com]

Thanks Tobias!

dl


On 12/4/17 11:07 PM, Tobias Hartmann wrote:
> Hi Dean,
>
> this looks good to me!
>
> Thanks,
> Tobias
>
> On 04.12.2017 18:44, 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