[PATCH] 8217561 : X86: Add floating-point Math.min/max intrinsics, approval request
vladimir.kozlov at oracle.com
Sat Feb 16 01:48:37 UTC 2019
Eric C. asked to add JMH microbanchmark to show performance of new code. You may be missed his e-mail:
We need to see benefits from these changes and verify them before pushing it.
On 2/15/19 12:35 PM, B. Blaser wrote:
> Thanks for your approval Nils and for the other reviews from Vladimir
> and Sandhya.
> Are we waiting for some more benchmarks or can I already push Jatin's
> contribution like this (webrev.04)?
> On Fri, 15 Feb 2019 at 14:33, Nils Eliasson <nils.eliasson at oracle.com> wrote:
>> Looks good!
>> What benchmark did you use? Is it in the microbenchmark suite?
>> On 2019-02-15 12:49, Viswanathan, Sandhya wrote:
>>> Hi Vladimir/Nils, Could one of you please review and approve the patch? You have looked at the previous versions of this patch so hoping it will be a quick review for you.
>>> The updated patch from Bernard looks ok to me. I am not an official reviewer though.
>>> Best Regards,
More information about the hotspot-compiler-dev