[7u-dev] Request for approval 8029302: Performance regression in Math.pow intrinsic

2014-08-21 Thread Sergey Gabdurakhmanov
Hello, Could you please approve the back-port of the fix for 8029302 to jdk7u-dev? The bug: https://bugs.openjdk.java.net/browse/JDK-8029302 The webrev: http://cr.openjdk.java.net/~sgabdura/8029302/webrev.00/ The review threads: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-Ap

Re: [7u-dev] Request for approval 8029302: Performance regression in Math.pow intrinsic

2014-08-21 Thread Seán Coffey
Approved. Please add a suitable noreg- label to bug report. regards, Sean. On 21/08/2014 10:44, Sergey Gabdurakhmanov wrote: Hello, Could you please approve the back-port of the fix for 8029302 to jdk7u-dev? The bug: https://bugs.openjdk.java.net/browse/JDK-8029302 The webrev: http://cr.ope

Re: [7u-dev] Request for approval 8029302: Performance regression in Math.pow intrinsic

2014-08-21 Thread Niclas Adlertz
Hi, Don't forget to also backport 8042052. It fixes an issue introduced by 8029302. Kind Regards, Niclas Adlertz On 08/21/2014 12:20 PM, Seán Coffey wrote: Approved. Please add a suitable noreg- label to bug report. regards, Sean. On 21/08/2014 10:44, Sergey Gabdurakhmanov wrote: Hello,

[7u-dev] Request for Approval and Review: 6883953: java -client -XX:ValueMapInitialSize=0 crashes

2014-08-21 Thread Vladimir Kempik
Hello! I'd like to ask an approval to backport 6883953 into 7u-dev. The patch does not apply cleanly, the required modifications are quite small. I will need a review for this. The webrev for jdk7: http://cr.openjdk.java.net/~vkempik/6883953/webrev.00/ The difference, compared to jdk9 is the c

[7u-dev] Request for approval 8042052: assert(t != NULL) failed: must set before get

2014-08-21 Thread Sergey Gabdurakhmanov
Hello, Could you please approve the back-port of the fix for 8042052 to jdk7u-dev? It's a patch on top of backport of 8029302 and I'll commit both patches in one push. The bug: https://bugs.openjdk.java.net/browse/JDK-8042052 The webrev: http://cr.openjdk.java.net/~sgabdura/8042052/webre

Re: [7u-dev] Request for Approval and Review: 6883953: java -client -XX:ValueMapInitialSize=0 crashes

2014-08-21 Thread Seán Coffey
Approved. Please get a peer code review before pushing. regards, Sean. On 21/08/2014 12:46, Vladimir Kempik wrote: Hello! I'd like to ask an approval to backport 6883953 into 7u-dev. The patch does not apply cleanly, the required modifications are quite small. I will need a review for this.

Re: [7u-dev] Request for approval 8042052: assert(t != NULL) failed: must set before get

2014-08-21 Thread Seán Coffey
Another bug that needs a noreg- label... Can you add one to both of the bugs that you've raised today ? Approved. regards, Sean. On 21/08/2014 16:54, Sergey Gabdurakhmanov wrote: Hello, Could you please approve the back-port of the fix for 8042052 to jdk7u-dev? It's a patch on top of backpo

Re: [7u-dev] Request for approval 8042052: assert(t != NULL) failed: must set before get

2014-08-21 Thread Sergey Gabdurakhmanov
Sure. I'll do it before push. BR, Sergey - Исходное сообщение - От: sean.cof...@oracle.com Кому: sergey.gabdurakhma...@oracle.com, albert.n...@oracle.com, niclas.adle...@oracle.com, jdk7u-dev@openjdk.java.net Отправленные: Четверг, 21 Август 2014 г 20:24:26 GMT +04:00 Абу-Даби, Маскат Те