RFR(M) 8138892: C1: Improve counter overflow checking

[Christian Thalinger]

Sorry, I’m pretty busy but I will try to sponsor this.

> On Oct 11, 2015, at 10:58 PM, Lindenmaier, Goetz <goetz.lindenmaier at sap.com> wrote:
> 
> Hi Martin,
>  
> this change looks good, thank you.
>  
> Could someone please sponsor this?
>  
> Best regards,
>   Goetz.
>  
> From: hotspot-compiler-dev [mailto:hotspot-compiler-dev-bounces at openjdk.java.net]On Behalf Of Doerr, Martin
> Sent: Mittwoch, 7. Oktober 2015 23:02
> To: hotspot compiler; Andrew Haley (aph at redhat.com)
> Subject: RFR(M) 8138892: C1: Improve counter overflow checking
>  
> Hi,
>  
> C1 currently loads the Method pointer in the fast path even though it is only required in the overflow case. In addition, stupid code gets generated when the frequency is set to 0 which occurs with -Xcomp.
>  
> The following webrev optimizes that:
> http://cr.openjdk.java.net/~mdoerr/8138892_c1_counter_overflow/webrev.02 <http://cr.openjdk.java.net/~mdoerr/8138892_c1_counter_overflow/webrev.02>
>  
> It’s not tested on aarch64.
>  
> Please review this change.  I need a sponsor, please.
>  
> Best regards,
>   Martin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20151020/6b43bc4c/attachment-0001.html>