RFR(M): 8136525 Generate interpreter entries only once and avoid unnecessary jump to jump

[Doerr, Martin]

Hi Coleen,

the new entries from the change "8134553: CRC32C implementations for x86/x64 targets" should also be improved like the other entries.
The add-on for the new CRC32C entries (x86 template interpreter) is included in this webrev:
http://cr.openjdk.java.net/~mdoerr/8136525_interp_entries/webrev.03/

Best regards,
  Martin


-----Original Message-----
From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] On Behalf Of Coleen Phillimore
Sent: Mittwoch, 16. September 2015 20:38
To: hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR(M): 8136525 Generate interpreter entries only once and avoid unnecessary jump to jump


This looks like a nice cleanup and I can sponsor it.

Can someone test this on aarch64?

http://cr.openjdk.java.net/~mdoerr/8136525_interp_entries/webrev.00/src/share/vm/interpreter/interpreter.cpp.html

Can you add a comment before 603 that you are reusing native or normal 
entry point addresses so you assume that they are generated first.

Coleen

On 9/16/15 4:44 AM, Doerr, Martin wrote:
> Hi,
>
> I improved the interpreter entries by avoiding multiple generation of the normal and native entries as well as entries which just jump to one of them.
> Please note that I didn't test it on aarch64.
>
> Please review this change.  I please need a sponsor.
> http://cr.openjdk.java.net/~mdoerr/8136525_interp_entries/webrev.00/
>
> Best regards,
>    Martin
>