RFR(S) 8166750: profiling handles statically bindable call sites differently than the interpreter

Igor Veresov igor.veresov at oracle.com
Wed Oct 25 22:16:47 UTC 2017

Sure. I’ve updated the webrev: http://cr.openjdk.java.net/~iveresov/8166750/webrev.02/ <http://cr.openjdk.java.net/~iveresov/8166750/webrev.02/>
Also added a comment in HotSpotMethodData.java per Doug’s request.


> On Oct 25, 2017, at 10:29 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> Igor
> Can you factor out checks into boolean function in shared place? May be move some surrounding code into it too - I see the same code on all platforms.
> Thanks,
> Vladimir
> On 10/24/17 8:52 PM, Igor Veresov wrote:
>> This a fix from Tom that I ported to all architectures and the new repo structure. While that fix doesn’t not solve the problem of the interpreter-C1 profiling style discrepancy completely it speeds up profiling of the statically bindable call sites and we’d like to push that. I also added a bit of a code to JVMCI to do the profile fix up analogous to what happens in CI.
>> Webrev: http://cr.openjdk.java.net/~iveresov/8166750/webrev.01/
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8166750
>> Thanks,
>> igor

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20171025/95cad540/attachment.html>

More information about the hotspot-compiler-dev mailing list