RFR(XXL): 8144019: PPC64 C1: Introduce Client Compiler

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Thu Nov 26 09:33:30 UTC 2015

Hi Martin,

thanks for doing this change, and especially for doing all
the implementation of the C1 port!

I only have a few comments:

load_const32() should be defined in macroAssembler_ppc.hpp, not in
assembler_ppc.hpp. (Especially after you added the comment in

I does not make sense to use flag CommentedAssembly and #ifndef PRODUCT here.
Block_comment() is stubbed out in product builds. CommentedAssembly is constant false
in opt builds.  Why would you want to switch it off in debug builds?
I think this is meant to protect additional code as in
macroAssembler_sparc.cpp:1298, "snprintf(buffer, sizeof(buffer), "verify_oop at %d", offset());"
so that it's optimized out in product builds.

File c1_FpuStackSim_ppc.cpp is not needed, it's empty.
Please delete it.

Best regards,

From: hotspot-compiler-dev [mailto:hotspot-compiler-dev-bounces at openjdk.java.net] On Behalf Of Doerr, Martin
Sent: Mittwoch, 25. November 2015 15:06
To: hotspot-compiler-dev at openjdk.java.net
Subject: RFR(XXL): 8144019: PPC64 C1: Introduce Client Compiler


we would like to contribute our PPC64 port of the Client Compiler to support Tiered Compilation.
The change includes refactoring of some functionality which is shared between C1 and C2 and some updates.

The webrev is here:

It only changes PPC64 files, with one minor exception: make/linux/Makefile

Please review. I will also need a sponsor, please.

Best regards,

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

More information about the hotspot-compiler-dev mailing list