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

[Tobias Hartmann]

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