RFR (S): add CodeComments functionality to assember stubs (original) (raw)
Vladimir Kozlov vladimir.kozlov at oracle.com
Thu Sep 20 14:54:09 PDT 2012
- Previous message: RFR (S): add CodeComments functionality to assember stubs
- Next message: RFR (S): add CodeComments functionality to assember stubs
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
I think it is good and I will push it if nobody objects.
Thanks, Vladimir
Lindenmaier, Goetz wrote:
Hi,
The Assembler and CodeBuffer classes supply CodeComment / blockcomment() functionality, which does not work with stubs. The comments are not printed with +PrintStubCode or +PrintInterpreter because the comments are lost when the code is turned into a Stub, while they are kept if the code is copied to a CodeBlob. We fixed this in our SAP JVM, and contributed the change to the ppc-aix-port some while ago, see http://hg.openjdk.java.net/ppc-aix-port/jdk7u/hotspot/rev/d65d0876ab43. I propose to add this fix to the OpenJDK mainline. A webrev can be found here: http://cr.openjdk.java.net/~goetz/webrevs/webrev-commentsinstubs/ Basically the change passes the codeBuffer to the Stub constructor, and adapts the disassembler to print the comments. In the debug build the InterpreterCodelet Stub has a new field holding the code comments. I also added some ttyLocks and \ns <file:///\ns> to beautify the output. Could somebody please create a bug id for this issue and review the changes? Thank you and best regards, Goetz
- Previous message: RFR (S): add CodeComments functionality to assember stubs
- Next message: RFR (S): add CodeComments functionality to assember stubs
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
More information about the hotspot-compiler-dev mailing list